User tests: Successful: Unsuccessful:
Pull Request for Issue #25611 .
Added css to separate buttons as per Issue #25611. Currently unsure about any extant convention for positioning within file. Have tried to avoid the obvious elements that should be near the top of the file.
fields should have some space
fields should have some space
None
Status | New | ⇒ | Pending |
Category | ⇒ | Administration Templates (admin) |
See this doc for instructions https://docs.joomla.org/Joomla_LESS/en
Ah I had my suspicions about that but couldn't be sure. Will amend.
Also, a general margin will add 10px of space in each direct. Instead I'd suggest adding a bottom margin to the parent container
How about a left/right margin of 2px as the buttons do seem to be right up against each other also.
Propose to place these changes into /administrator/templates/isis/less/blocks/_media.less
Labels |
Added:
?
|
I have tested this item
I would recommend instead of using 2px to use .25em instead. This is eqv to a "space" char, and being that its fluid will scale to other changes.
It is fine as px. Everything else in this template is in px. Just changing this one value to em wont have the benefit you expect - you would need to convert everything
This breaks button group display. Margin should be applied to the form, not to the buttons. Maybe apply custom class to folder form, like .uploadform
is applied to the upload form right above the folder form?
When you say button group do you mean the fact that the buttons are paired and touching like elsewhere in the backend for yes/no buttons etc?
If we agree that these buttons should be treated in the same way then Id think just removing the margin-right will get that job done..
@brianteeman actually thats not true, we specifically looked at both with the icons. They were changed from px to em ( specifically a single space ) which solved the issues. The point I think you missed is, .25em = char $32 which 2px does not.
Category | Administration Templates (admin) | ⇒ | Administration com_media Templates (admin) |
@brianteeman before you continue to spout on about what you clearly don't comprehend please go get educated. Your demeaning me on a public platform when you haven't even taken the time to familiarize yourself ACCURATELY with my statements.
As you pointed out a px is a finite measurEment COMPLETELY unrelated to the content and wholy related to the device. Ergo 2px on any given display may/may not be of different size.
em is a relative unit. The "space" character most often used on websites is roughly eqv to 2.5px. This is not possible for a ton of reasons so you must look to using some other form of measurement ( of which there are many )
turns out ( as we found out doing the icon fixing ) that .25em is the perfect size to emulate a "space" character.
Since this is a fluid display that we're working on we MUST move away from fixed sizing like was erroneously done decades ago.
With such a vast array of displays that we now have we have no clue what they will be used/viewed on. Ergo the push to responsive web design.
Your statement that "this is a px template" is prima facie true it is also false. It's a bootstrap based template which is a FLUID template! Because of convention it uses .px but its behavior's are primarily controlled by %'s & em's.
Lastly just because a method has been used in the past doesn't make it the best way.
But who am I to want to actually REMOVE a bug instead of trading one for another
edit: Just incase anyone doubts my statement he's the padding actually at work as it has been for a long time
If writing that makes you happy then fine. It's factually and technically incorrect and based on many assumptions and misunderstandings, not least a failure to understand my original comment. I suggest you educate yourself better on what an em unit is
I never said 2px = .25em, I said .25em = chr$32! .25em is closer to 3px then 2px lol
I have tested this item
@gogicomputers can you please retest?
I have tested this item
@infograf768 does this need a second retest?
not necessary. can be set rtc
Status | Pending | ⇒ | Ready to Commit |
Status "Ready To Commit".
Thank you for making Joomla! more pretty.
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-08-08 11:31:06 |
Closed_By | ⇒ | HLeithner | |
Labels |
Added:
?
|
You need to update the LESS files and compile