search

Piwigo / Piwigo-Android

Piwigo Native Android App
GNU General Public License v3.0
141 stars 43 forks source link

Highlight the current account #135

Closed ramack closed 4 years ago

ramack commented 4 years ago

The activity "Manage Accounts" allows to select the current account and modify the selected on. But the selected one is not visually highlighted. This shall be added.

Valou447 commented 4 years ago

Actually, the selected one is highlighted, but when you switch to another account, the old one is still colored like if it was also selected.

Leaving and getting back to the activity fixes the "display" issue.

Then we need to find a way to refresh to UI properly.

ramack commented 4 years ago

This is roughly how I had it in mind as it was. But today I do not see any selection at all on my phone...

Valou447 commented 4 years ago

After discussing this with @plegall , we thought that having a RadioButton would be a good way to inform the user what account is used (for instance, if the user has 2 accounts, we may ask which one is used, the one which is darker or the light gray one)..

So with a few modifications of the account_row.xml, I managed to do something like this:

PREVIEW

How do you feel about it ? Does it seems like a good alternative to you ?

ramack commented 4 years ago

That's great for sure! Go for it!

Valou447 commented 4 years ago

WIP, see f47d83a88f4e80eaad4aed9eacf7eb4bf85d3548

I still need to figure out the proper way to refresh the ListView without refreshing the whole activity, if it is possible, and I guess it is.