? ? Pending

User tests: Successful: Unsuccessful:

avatar coolcat-creations
coolcat-creations
29 Aug 2019

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:
grafik

RTL, regular Bootstrap without icons inside:
grafik

Problems appear here:
The Action Button is broken although it should have the same appeareance like the save and close button:
grafik
grafik

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:
grafik
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:

  • when they use standard bootstrap classes OR
  • at least one standard of Joomla Button Code Style

grafik

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.

avatar coolcat-creations coolcat-creations - open - 29 Aug 2019
avatar coolcat-creations coolcat-creations - change - 29 Aug 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Aug 2019
Category Unit Tests Repository Administration
avatar coolcat-creations coolcat-creations - change - 29 Aug 2019
The description was changed
avatar coolcat-creations coolcat-creations - edited - 29 Aug 2019
avatar coolcat-creations coolcat-creations - change - 29 Aug 2019
Labels Added: ? ?
avatar Quy
Quy - comment - 29 Aug 2019

Please commit to 4.0-dev branch.

avatar joomla-cms-bot joomla-cms-bot - change - 29 Aug 2019
Category Unit Tests Repository Administration Administration Templates (admin)
avatar zero-24
zero-24 - comment - 29 Aug 2019

@coolcat-creations I have just moved the branch against 4.0-dev instead of staging so this can actually be merged ;)

avatar coolcat-creations
coolcat-creations - comment - 29 Aug 2019

@zero-24 thank you - but first the button-code needs to be fixed somehow...

avatar brianteeman
brianteeman - comment - 29 Aug 2019

would be better for you to create issues for the buttons that need fixing

avatar Hackwar
Hackwar - comment - 29 Aug 2019

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...

avatar coolcat-creations
coolcat-creations - comment - 29 Aug 2019

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

grafik

vs
grafik

avatar brianteeman
brianteeman - comment - 29 Aug 2019

All these buttons have to have an icon?

avatar coolcat-creations
coolcat-creations - comment - 29 Aug 2019

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.

avatar brianteeman
brianteeman - comment - 29 Aug 2019

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

avatar coolcat-creations
coolcat-creations - comment - 29 Aug 2019

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.
grafik

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.

grafik

btw forget about my changes for the radio switch they need to be done different definetly :-)

avatar mbabker
mbabker - comment - 29 Aug 2019

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.

avatar brianteeman
brianteeman - comment - 29 Aug 2019

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

avatar coolcat-creations
coolcat-creations - comment - 29 Aug 2019

Ok, waiting then for the decision from the design team. I think it looks too colorful and chaotic when all those different button styles are used. Also the current ones (in the plain variation) don't work with icons inside.
grafik

avatar brianteeman
brianteeman - comment - 29 Aug 2019

Also the current ones (in the plain variation) don't work with icons inside.

Not true

avatar coolcat-creations
coolcat-creations - comment - 29 Aug 2019

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?

avatar coolcat-creations
coolcat-creations - comment - 29 Aug 2019

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...

avatar brianteeman
brianteeman - comment - 29 Aug 2019

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

avatar coolcat-creations
coolcat-creations - comment - 29 Aug 2019

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.

avatar brianteeman
brianteeman - comment - 29 Aug 2019

And where is this feedback. You keep mentioning it but I don't believe it has ever been made public.

avatar mbabker
mbabker - comment - 29 Aug 2019

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.

avatar C-Lodder
C-Lodder - comment - 30 Aug 2019

One thing that also caught my eye was that the:

  1. Warning / Danger button colours look similar
  2. Primary / Secondary colours look very similar

With that in mind, you may as well remove them from the colour array map. Or make them more distinguishable.

avatar coolcat-creations
coolcat-creations - comment - 30 Aug 2019

@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.

avatar coolcat-creations coolcat-creations - change - 30 Aug 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-08-30 08:25:50
Closed_By coolcat-creations
Labels Added: ?
Removed: ?
avatar coolcat-creations coolcat-creations - close - 30 Aug 2019
avatar brianteeman
brianteeman - comment - 30 Aug 2019

Pretty sure I never said anything about a design team as I have no knowledge of one

avatar coolcat-creations
coolcat-creations - comment - 30 Aug 2019

Sorry, was a mistake, @mbabker mentioned a design team.

Add a Comment

Login with GitHub to post a comment