Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Few issues with the new DNS sub-menu section #1374

Closed
vozersky opened this issue Aug 15, 2017 · 15 comments
Closed

Few issues with the new DNS sub-menu section #1374

vozersky opened this issue Aug 15, 2017 · 15 comments

Comments

@vozersky
Copy link
Member

vozersky commented Aug 15, 2017

I suggest to keep it as one issue.

  1. Scrollbar is missing at both DNS server lists (regular and DNScrypt)

  1. We should change wording a little bit @vbagirov
Screenshot:

image


"Custom DNS server" doesn't look like an option to click on, more like an info tab, let's at least name it "Choose a DNS server" or something like this ---

  1. Russian localization issues @zebrum

  1. UI glitch when 2 DNS servers are "active" @nkartyshov
Screenshot:

image

  1. In System default section text is displaced:
Screenshot:

photo_2017-08-31_14-33-03

@vozersky vozersky added the UI label Aug 15, 2017
@vbagirov
Copy link
Member

@vozersky no.2 - fixed in oneskyapp.

@nkartyshov
Copy link
Contributor

no.4 fixed, review AFA-CR-80

@vozersky vozersky added this to the 2.10 milestone Aug 16, 2017
@ameshkov ameshkov assigned nkartyshov and ameshkov and unassigned ameshkov Aug 23, 2017
@zebrum
Copy link
Member

zebrum commented Aug 31, 2017

added №5
@nkartyshov please look at it

@zebrum
Copy link
Member

zebrum commented Aug 31, 2017

And it seems that №4 is not fixed yet:
@nkartyshov

photo_2017-08-31_14-51-33

@zebrum
Copy link
Member

zebrum commented Sep 6, 2017

no.3 fixed in ru localization.

@nkartyshov
Copy link
Contributor

no. 4 fixed. Can we close the issue?

@zebrum
Copy link
Member

zebrum commented Sep 15, 2017

@nkartyshov what about №5?

@nkartyshov
Copy link
Contributor

Sorry, no.5 fixed

@vozersky
Copy link
Member Author

#1443

@vozersky vozersky reopened this Sep 22, 2017
@vozersky
Copy link
Member Author

This issue still needs some love:


  1. Scrollbar is missing at both DNS server lists (regular and DNScrypt)

@nkartyshov
Copy link
Contributor

@vozersky fixed

@vozersky
Copy link
Member Author

vozersky commented Oct 30, 2017

The visual glitch with 2 active DNS servers (usual one and DNSCrypt) us back

@nkartyshov please assist

@vozersky vozersky reopened this Oct 30, 2017
@nkartyshov
Copy link
Contributor

Fixed, adguard-android/pull-requests/187

@Heresh83
Copy link

`

@Heresh83
Copy link

ثابت ، adguard-android / pull-applications / 187

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants