User tests: Successful: Unsuccessful:
This PR is about to fix the styling of the buttons for rtl and ltr. But it cannot be tested yet because there is some wrong code in the backend which needs to be fixed first.
LTR, regular Bootstrap without icons inside:
RTL, regular Bootstrap without icons inside:
Problems appear here:
The Action Button is broken although it should have the same appeareance like the save and close button:
The reason for it, is that the Action Button has a completely different semantic than the save and close button. It should be fixed in the code first.
Next Problem appear here:
There are 3 Buttons bigger than the other ones. The simple reason is, that ones are using btn-sm class and the others not. The classes should be unified to make the buttons appear the same. It does not make sense to make big buttons smaller or small buttons bigger just to fix this. They should have the same button size class.
This PR unifies the styling of all buttons accross the backend:
Buttons that stil appear different have to be fixed from a technical perspektive.
After the code part is fixed it may be that this PR needs some small refinement, but first the code needs to be fixed.
Status | New | ⇒ | Pending |
Category | ⇒ | Unit Tests Repository Administration |
Labels |
Added:
?
?
|
Category | Unit Tests Repository Administration | ⇒ | Administration Templates (admin) |
@coolcat-creations I have just moved the branch against 4.0-dev instead of staging so this can actually be merged ;)
would be better for you to create issues for the buttons that need fixing
Can you please elaborate what you mean with buttons being broken? Looking at the code, the buttons are pretty much identical, except that the dropdown buttons are wrapped in a div...
Can you please elaborate what you mean with buttons being broken? Looking at the code, the buttons are pretty much identical, except that the dropdown buttons are wrapped in a div...
Well I wrote it in the PR at the top... I added it also the the issue, hope that helps.
The Action Drop Down and the Module dropdown are different from save and close
All these buttons have to have an icon?
All these buttons have to have an icon?
no they don't have to have an icon, but the install buttons on the control panel have so I suppose to be consistent in the types of buttons. If there is a save button with an icon, all save buttons should have this icon. If the install button has one, all install buttons should have the same. That UX...
If few buttons use btn-sm classes in the toolbar, the buttons in the toolbar should either use or not use this sizing class.
If there is a save button with an icon, all save buttons should have this icon. If the install button has one, all install buttons should have the same. That UX...
Of course that is good ux but we have to be able to support extensions that may not have defined an icon for everything. And in many places there is no need for an icon - example the Comment button here on github, the login button in the joomla admin, the documentation link in template styles.
I dont see any solid colour buttons in your first post. So there are no button styles that can be used. You have to have a button in white with a colour bar with an icon. This was not part of your design proposal
This is how regular bs buttons look like, they always look according to the style of the Joomla Buttons. Does not matter that they dont have icons. If they have Icons, the icons will appear in the squared colored area of the button. If there are no icons these squares a empty. this will unify the style of the template.
In the current 4.0dev the regular buttons look completely different from the ones used in Joomla, I think they should have the same appereance. Also if you add an icon to it they don't work. And don't work rtl with an icon inside. This PR here is fixing this. I think if I follow the suggestions from Charlie I can redo it in a better way but as said, the code has to be fixed still.
btw forget about my changes for the radio switch they need to be done different definetly :-)
It’s only unifying the button component use if every button has an icon though. Otherwise, to me all of those buttons look broken as if something is missing. If this is the design team coming down with a mandate that every button will have an icon, then so be it, just another decision people are going to either live with or hack around.
Consistency is good, but I feel like there needs to be two supported variations with this design; a button with an icon (which is what the current style is and fits the majority of use cases), and a “plain” style which is the “classic” full color buttons.
This is how regular bs buttons look like,
No they have no icons and coloured backgrounds not empty squares
This was not part of your design proposal
Also the current ones (in the plain variation) don't work with icons inside.
Not true
No they have no icons and coloured backgrounds not empty squares
There is a misunderstanding. I meant how they look like with this PR. Without this PR they are solid colored and that does not fit to the template. But ok, who is in charge of design currently, maybe we can ask the design team?
Also the current ones (in the plain variation) don't work with icons inside.
Not true
See my screenshot, thats not how it looks nice... And if they have an icon they should look like the Joomla default ones...
Without this PR they are solid colored and that does not fit to the template.
What do you mean by does not fit the template - thats what they were in your design proposal
And if they have an icon
Without this PR they are solid colored and that does not fit to the template.
What do you mean by does not fit the template - thats what they were in your design proposal
Due to user feedback they were made less bold / screaming than in the design proposal from 2 years ago... so the style should be unified through the template. All the Joomla buttons have been redesigned and are changed. Why should the others be different than the regular Joomla Buttons. It makes the backend chaotic and less classy.
And where is this feedback. You keep mentioning it but I don't believe it has ever been made public.
Maybe I’m as blind as I’m considered a troll, but in all honesty the buttons in your screenshot look broken without icons to me. That button style comes across as needing an icon to make it look right, it all looks off to me with the empty squares.
Defaulting to the icon style is OK. Add a btn-noicon-*
variant that takes buttons back to the default full color design (the same way Bootstrap ships the btn-outline-*
variant) that way someone has to put in the extra effort to use that style but it’s still supported. I do think the icon style is going to be the primarily used style but I don’t think the icon style needs to be hardcoded and unavoidable. That’s my only concern with this, not that the design is bad but that it’s forcing specific design elements to always be used when they aren’t needed.
One thing that also caught my eye was that the:
With that in mind, you may as well remove them from the colour array map. Or make them more distinguishable.
@C-Lodder I'd wait for a suggestion from the design team @brianteeman mentioned. IMHO all the Buttons in backend should follow an equal styling. Not some solid, some white with square color field at the front and others again different. Closing this PR for now until we have a design decision. The issue connected to this should stay open because it's about different markups and this should definetly be fixed.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-08-30 08:25:50 |
Closed_By | ⇒ | coolcat-creations | |
Labels |
Added:
?
Removed: ? |
Pretty sure I never said anything about a design team as I have no knowledge of one
Please commit to
4.0-dev
branch.