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.

Sign up for GitHub

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

Jump to bottom

Top Clients small view #6338

Closed
4 tasks done
TheCeet opened this issue Oct 21, 2023 · 4 comments
Closed
4 tasks done

Top Clients small view #6338

TheCeet opened this issue Oct 21, 2023 · 4 comments
Assignees
@IldarKamalov
Labels
enhancement P4: Low UI
Milestone
v0.107.41

Comments

@TheCeet
Copy link

TheCeet commented Oct 21, 2023

Prerequisites

Platform (OS and CPU architecture)

Linux, ARMv7

Installation

GitHub releases or script from README

Setup

On one machine

AdGuard Home version

0.107.40

Action

Update to v0.107.40

Expected result

Wider text area, name of the client like the past.

Actual result

Since release 0.107.40, the ‘Top clients’ name view on a mobile device is not that clear.
Device: iPhone 13 Pro, Safari.
see:
IMG_5246

Additional information and/or screenshots

No response
@ainar-g ainar-g added this to the v0.107.41 milestone Oct 23, 2023
adguard pushed a commit that referenced this issue Nov 3, 2023
@IldarKamalov @EugeneOne1
Pull request 2055: ADG-7651 fix dashboard client cell ellipsis
cbcc17a 
Merge in DNS/adguard-home from ADG-7651 to master

Updates #6338.

Squashed commit of the following:

commit 21acb7a
Merge: 1c0ba2c 6a3906a
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Fri Nov 3 17:12:55 2023 +0300

    Merge branch 'master' into ADG-7651

commit 1c0ba2c
Author: Ildar Kamalov <ik@adguard.com>
Date:   Thu Nov 2 10:06:48 2023 +0300

    changelog

commit 14d8a2c
Author: Ildar Kamalov <ik@adguard.com>
Date:   Tue Oct 31 18:04:18 2023 +0300

    fix changelog

commit bfc3f45
Author: Ildar Kamalov <ik@adguard.com>
Date:   Tue Oct 31 18:03:24 2023 +0300

    changelog

commit 64cc673
Author: Ildar Kamalov <ik@adguard.com>
Date:   Tue Oct 31 18:00:50 2023 +0300

    ADG-7651 fix dashboard client cell ellipsis
@EugeneOne1
Copy link
Member

@TheCeet, hello and apologies for late response. We've improved the Top Clients' names appearance in the latest edge, could you please check it out?

@TheCeet
Copy link
Author

TheCeet commented Nov 3, 2023

@EugeneOne1 I’m unable to install edge builds. You may close this.
When next update is available, I’ll check it and re-open it if necessary.
Thanks in advance for the fix!

@ainar-g ainar-g modified the milestones: v0.107.42, v0.107.41 Nov 13, 2023
@TheCeet
Copy link
Author

TheCeet commented Nov 13, 2023

IMG_5336
@EugeneOne1
I updated to v0.107.41.
It's a little better but still (my opinion) the request count field is wide. And I think it could shrink a little to fit it better.

Here, I edited the width which is perfect for me:
IMG_5337

@ainar-g
Copy link
Contributor

ainar-g commented Nov 14, 2023

@TheCeet, please file a separate issue about that proposal.

annguyen0 pushed a commit to annguyen0/AdGuardHome that referenced this issue Nov 27, 2023
@IldarKamalov @annguyen0
Pull request 2055: ADG-7651 fix dashboard client cell ellipsis
af8772b 
Merge in DNS/adguard-home from ADG-7651 to master

Updates AdguardTeam#6338.

Squashed commit of the following:

commit 21acb7a
Merge: 1c0ba2c 6a3906a
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Fri Nov 3 17:12:55 2023 +0300

    Merge branch 'master' into ADG-7651

commit 1c0ba2c
Author: Ildar Kamalov <ik@adguard.com>
Date:   Thu Nov 2 10:06:48 2023 +0300

    changelog

commit 14d8a2c
Author: Ildar Kamalov <ik@adguard.com>
Date:   Tue Oct 31 18:04:18 2023 +0300

    fix changelog

commit bfc3f45
Author: Ildar Kamalov <ik@adguard.com>
Date:   Tue Oct 31 18:03:24 2023 +0300

    changelog

commit 64cc673
Author: Ildar Kamalov <ik@adguard.com>
Date:   Tue Oct 31 18:00:50 2023 +0300

    ADG-7651 fix dashboard client cell ellipsis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@IldarKamalov IldarKamalov

Labels
enhancement P4: Low UI
Projects
None yet
Milestone
v0.107.41
Development

No branches or pull requests
4 participants
@IldarKamalov @ainar-g @EugeneOne1 @TheCeet