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

No text on buttons on preferences page #16647

Closed
OlafvdSpek opened this issue Feb 15, 2021 · 8 comments
Closed

No text on buttons on preferences page #16647

OlafvdSpek opened this issue Feb 15, 2021 · 8 comments
Assignees
Labels
Bug A problem or regression with an existing feature newbie ui Issues relating to the user interface
Projects
Milestone

Comments

@OlafvdSpek
Copy link

OlafvdSpek commented Feb 15, 2021

5.1.0-rc2

The icons look pretty, but I don't really know what each icon does without the text..

image

https://demo.phpmyadmin.net/master-config/index.php?route=/preferences/manage&lang=nl

$cfg['ActionLinksMode'] = 'icons';
@williamdes williamdes added the ui Issues relating to the user interface label Feb 15, 2021
@williamdes williamdes added this to the 5.1.0 milestone Feb 15, 2021
@ibennetch ibennetch modified the milestones: 5.1.0, 5.1.1 Feb 24, 2021
@williamdes
Copy link
Member

Not sure if this is a bug. You can use tooltips.
image

@williamdes williamdes removed this from the 5.1.1 milestone Mar 14, 2021
@williamdes williamdes added this to Needs triage in issues via automation Mar 14, 2021
@williamdes williamdes moved this from Needs triage to To be sorted in issues Mar 14, 2021
@ibennetch
Copy link
Member

ibennetch commented Mar 15, 2021

@OlafvdSpek I'm not sure what your point is — the default of is to have icons+text ('both'), so a user would have to intentionally choose to switch to 'icons'. We've tried to keep some consistency with the icons to their use across the rest of the application (SQL queries, import, export, etc), so even though for instance the "SQL queries" icon isn't very intuitive, it matches the icon used for the SQL tab so anyone working in 'icon' mode should know what they mean.

What solution do you propose?

@ibennetch
Copy link
Member

Or perhaps your point is that this setting shouldn't affect these icons, which I agree with.

This is currently affected by $cfg['ActionLinksMode'], but should be $cfg['TabsMode'] instead.

@williamdes williamdes self-assigned this Mar 21, 2021
@williamdes williamdes changed the title No text on buttons No text on buttons on preferences page Mar 21, 2021
@williamdes williamdes added the Bug A problem or regression with an existing feature label Mar 21, 2021
@williamdes williamdes added this to the 5.1.1 milestone Mar 21, 2021
@williamdes
Copy link
Member

Or perhaps your point is that this setting shouldn't affect these icons, which I agree with.

This is currently affected by $cfg['ActionLinksMode'], but should be $cfg['TabsMode'] instead.

Thank you !
This will be fixed by 8c74a42

williamdes added a commit that referenced this issue Mar 21, 2021
Signed-off-by: William Desportes <williamdes@wdes.fr>
@OlafvdSpek
Copy link
Author

OlafvdSpek commented Mar 21, 2021

Thanks!

@williamdes
Copy link
Member

Thanks!

Let me know if you find more buttons affected

issues automation moved this from To be sorted to Closed Mar 22, 2021
@OlafvdSpek
Copy link
Author

Table - Search doesn't have text on the icons yet.

williamdes added a commit that referenced this issue Mar 26, 2021
Fixes: #16647

Signed-off-by: William Desportes <williamdes@wdes.fr>
@williamdes
Copy link
Member

Table - Search doesn't have text on the icons yet.

Indeed, fixed by ffb22f4

williamdes added a commit that referenced this issue Mar 27, 2021
Signed-off-by: William Desportes <williamdes@wdes.fr>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bug A problem or regression with an existing feature newbie ui Issues relating to the user interface
Projects
issues
  
Closed
Development

No branches or pull requests

3 participants