Changed handling of comments fallback dns

This commit is contained in:
Juan Gilsanz Polo 2024-02-29 15:17:27 +01:00
parent 133d29aa91
commit 7c0b592715
2 changed files with 84 additions and 13 deletions

View file

@ -1,5 +1,6 @@
// ignore_for_file: use_build_context_synchronously import 'dart:io';
import 'package:adguard_home_manager/screens/settings/dns/comment_modal.dart';
import 'package:flutter/material.dart'; import 'package:flutter/material.dart';
import 'package:provider/provider.dart'; import 'package:provider/provider.dart';
import 'package:flutter_gen/gen_l10n/app_localizations.dart'; import 'package:flutter_gen/gen_l10n/app_localizations.dart';
@ -48,13 +49,21 @@ class _FallbackDnsScreenState extends State<FallbackDnsScreen> {
final dnsProvider = Provider.of<DnsProvider>(context, listen: false); final dnsProvider = Provider.of<DnsProvider>(context, listen: false);
for (var item in dnsProvider.dnsInfo!.fallbackDns!) { for (var item in dnsProvider.dnsInfo!.fallbackDns!) {
if (item.contains("#")) {
fallbackControllers.add({
'comment': item
});
}
else {
final controller = TextEditingController(); final controller = TextEditingController();
controller.text = item; controller.text = item;
fallbackControllers.add({ fallbackControllers.add({
'controller': controller, 'controller': controller,
'error': null 'error': null,
'isComment': item.contains("#")
}); });
} }
}
validValues = true; validValues = true;
super.initState(); super.initState();
} }
@ -71,11 +80,16 @@ class _FallbackDnsScreenState extends State<FallbackDnsScreen> {
processModal.open(AppLocalizations.of(context)!.savingConfig); processModal.open(AppLocalizations.of(context)!.savingConfig);
final result = await dnsProvider.saveFallbackDnsConfig({ final result = await dnsProvider.saveFallbackDnsConfig({
"fallback_dns": fallbackControllers.map((e) => e['controller'].text).toList(), "fallback_dns": fallbackControllers.map(
(e) => e['controller'] != null
? e['controller'].text
: e['comment']
).toList(),
}); });
processModal.close(); processModal.close();
if (!context.mounted) return;
if (result.successful == true) { if (result.successful == true) {
showSnacbkar( showSnacbkar(
appConfigProvider: appConfigProvider, appConfigProvider: appConfigProvider,
@ -99,6 +113,37 @@ class _FallbackDnsScreenState extends State<FallbackDnsScreen> {
} }
} }
void openEditCommentModal(Map<String, dynamic> item, int position) {
if (width > 900 || !(Platform.isAndroid || Platform.isIOS)) {
showDialog(
context: context,
builder: (context) => CommentModal(
comment: item['comment'],
onConfirm: (value) {
setState(() => fallbackControllers[position] = { 'comment': value });
},
dialog: true,
),
);
}
else {
showModalBottomSheet(
context: context,
useRootNavigator: true,
builder: (context) => CommentModal(
comment: item['comment'],
onConfirm: (value) {
setState(() => fallbackControllers[position] = { 'comment': value });
},
dialog: false,
),
backgroundColor: Colors.transparent,
isScrollControlled: true,
isDismissible: true
);
}
}
return Scaffold( return Scaffold(
appBar: AppBar( appBar: AppBar(
title: Text(AppLocalizations.of(context)!.fallbackDnsServers), title: Text(AppLocalizations.of(context)!.fallbackDnsServers),
@ -168,29 +213,55 @@ class _FallbackDnsScreenState extends State<FallbackDnsScreen> {
child: Row( child: Row(
mainAxisAlignment: MainAxisAlignment.spaceBetween, mainAxisAlignment: MainAxisAlignment.spaceBetween,
children: [ children: [
Expanded( if (c['controller'] != null) Expanded(
child: TextFormField( child: TextFormField(
controller: c['controller'], controller: c['controller'],
onChanged: (value) => validateIp(c, value), onChanged: (value) => validateIp(c, value),
decoration: InputDecoration( decoration: InputDecoration(
prefixIcon: const Icon(Icons.dns_rounded), prefixIcon: Icon(
c['isComment'] == true
? Icons.comment_rounded
: Icons.dns_rounded
),
border: const OutlineInputBorder( border: const OutlineInputBorder(
borderRadius: BorderRadius.all( borderRadius: BorderRadius.all(
Radius.circular(10) Radius.circular(10)
) )
), ),
errorText: c['error'], errorText: c['error'],
labelText: AppLocalizations.of(context)!.dnsServer, labelText: c['isComment'] == true
? AppLocalizations.of(context)!.comment
: AppLocalizations.of(context)!.dnsServer,
) )
), ),
), ),
const SizedBox(width: 8), const SizedBox(width: 8),
if (c['comment'] != null) Expanded(
child: Row(
mainAxisAlignment: MainAxisAlignment.spaceBetween,
children: [
Text(
c['comment'],
style: TextStyle(
fontSize: 16,
color: Theme.of(context).listTileTheme.iconColor
),
),
IconButton(
onPressed: () => openEditCommentModal(c, fallbackControllers.indexOf(c)),
icon: const Icon(Icons.edit),
tooltip: AppLocalizations.of(context)!.edit,
)
],
),
),
IconButton( IconButton(
onPressed: () { onPressed: () {
setState(() => fallbackControllers = fallbackControllers.where((con) => con != c).toList()); setState(() => fallbackControllers = fallbackControllers.where((con) => con != c).toList());
checkValidValues(); checkValidValues();
}, },
icon: const Icon(Icons.remove_circle_outline) icon: const Icon(Icons.remove_circle_outline),
tooltip: AppLocalizations.of(context)!.remove,
) )
], ],
), ),

View file

@ -47,7 +47,7 @@ class _UpstreamDnsScreenState extends State<UpstreamDnsScreen> {
final dnsProvider = Provider.of<DnsProvider>(context, listen: false); final dnsProvider = Provider.of<DnsProvider>(context, listen: false);
for (var item in dnsProvider.dnsInfo!.upstreamDns) { for (var item in dnsProvider.dnsInfo!.upstreamDns) {
if (item == '#') { if (item.contains("#")) {
dnsServers.add({ dnsServers.add({
'comment': item 'comment': item
}); });