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

cleanup account switcher #3310

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

r10s
Copy link
Member

@r10s r10s commented Sep 20, 2024

do not show 'Add Profile' in bold.

this looks less cluttered
and makes the "bold" of the selected account more outstanding:

before, there were two types of "boldness",
the add profile button (using bold only)
and the selected profile (using bold and a background), this results in a bit cluttered, unsteady look,
esp. when there are few accounts and more bold than normal is shown.

also, this makes "bold" more outstanding,
as this attribute is given to one line only.
there is no need for the "add profile" to be that outstanding (in contrast to eg. the "add chat" dialog)

the "Add Profile" button with its "+" is still recognisable enough and is not really mixed with an existing profile. otherwise, we could add some more whitespace between the accounts and "add profile". btw: wondering if "Add Profile" shouln't be the last item

it is a minor, sure :)

before - a bit noisy abd kooking as if twi things are selected:

after - less agitated:

do not show 'Add Profile' in bold.

this looks less cluttered
and makes the "bold" of the selected account more outstanding:

before, there were two types of "boldness",
the add profile button (using bold only)
and the selected profile (using bold and a background),
this results in a bit cluttered, unsteady look,
esp. when there are few accounts and more bold than normal is shown.

also, this makes "bold" more outstanding,
as this attribute is given to one line only.
there is no need for the "add profile" to be that outstanding
(in contrast to eg. the "add chat" dialog)

it is a minor, sure :)
@r10s r10s requested a review from adbenitez September 20, 2024 13:06
Copy link

To test the changes in this pull request, install this apk:
📦 app-preview.apk

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

Successfully merging this pull request may close these issues.

1 participant