-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Query log UI issue on tablet device #4775
Comments
That not only for iPad, on the any mobile or even PC on small size webiste, when the feature column on the top left need to click to open, that would have that issue |
Updates #4775 Squashed commit of the following: commit 9ad85d2 Merge: 95aa29d 41f081d Author: Ildar Kamalov <ik@adguard.com> Date: Tue Aug 2 11:44:04 2022 +0300 Merge branch 'master' into 4775-popup commit 95aa29d Author: Ildar Kamalov <ik@adguard.com> Date: Mon Aug 1 16:21:23 2022 +0300 client: fix query log issue on tablet devices
Updates #4775 Squashed commit of the following: commit 9ad85d2 Merge: 95aa29d 41f081d Author: Ildar Kamalov <ik@adguard.com> Date: Tue Aug 2 11:44:04 2022 +0300 Merge branch 'master' into 4775-popup commit 95aa29d Author: Ildar Kamalov <ik@adguard.com> Date: Mon Aug 1 16:21:23 2022 +0300 client: fix query log issue on tablet devices
Hello, This is quite a bad UI change. Earlier I could tap the question glyphs to hover over the information. This modal pops up when I tap just about anywhere in Query Log. This is also quite buggy. If I simulate a tablet device on desktop, click anywhere, close the modal by pressing Esc and immediately hover over a question glyph, it shows the box. |
@agneevX, clicking outside of the modal closes it for me. Is that not the case for you? Although, I personally would also add an explicit ❌ button.
I cannot reproduce that. For me, it only opens when I click on a query log record. Can you file a new issue, including your OS/browser and the viewport size in the Additional section? Thanks. |
Updates AdguardTeam#4775 Squashed commit of the following: commit 9ad85d2 Merge: 95aa29d 41f081d Author: Ildar Kamalov <ik@adguard.com> Date: Tue Aug 2 11:44:04 2022 +0300 Merge branch 'master' into 4775-popup commit 95aa29d Author: Ildar Kamalov <ik@adguard.com> Date: Mon Aug 1 16:21:23 2022 +0300 client: fix query log issue on tablet devices
Actual Behavior
On iPad, Block button covers client name, and it is hard to press client IP or the name
Better use the same popup as on mobile phone design, when press/click the log item.
Screenshots
Screenshot:
Additional Information
The text was updated successfully, but these errors were encountered: