User tests: Successful: Unsuccessful:
Changed the styling of the sidebar to move it off the "title" baseline when "Display Header" is set to "No". Also added stylizing to the sidebar "arrow" to give it a little more visual presence: http://awesomescreenshot.com/0ef4ib4w7b
http://awesomescreenshot.com/0024ib411b
http://awesomescreenshot.com/0204ib7k8f
Labels |
Added:
?
|
Title |
|
Title |
|
Category | ⇒ | Templates (admin) |
This patch adds some space between toolbar & sidebar.
Before:
Sorry, I've added the images in the wrong order...
The 1st image is the extra space after the patch.
The 2nd image is before applying the patch.
@test
Tested it in Joomla 3.4
1) I can not simulate te problems of the "Display Header" i see in the added pictures (http://awesomescreenshot.com/0ef4ib4w7b | http://awesomescreenshot.com/0024ib411b | http://awesomescreenshot.com/0204ib7k8f).
2) I can see the change in the "top" styling of the left sidebar. Its now on the top in-line with the other collums, indeed after the patch the visual presence is better!
@test
Patch also adds a little styling to the arrow:
Status | Pending | ⇒ | Ready to Commit |
RTC based one various testing. Thanks for testing and coding!
Shouldn't the less file be updated as well? And what about the rtl version?
Status | Ready to Commit | ⇒ | Pending |
Thanks @losedk moving back to pending. can you generate the CSS files using the https://github.com/joomla/joomla-cms/blob/staging/build/generatecss.php and update the less files @ChristopherRaymond
As i have stated in issue #6178 (comment) i think changing the margin is not the best solution.
If an option in admin template is supposed to hide the header (title + joomla logo), is it needed to display the title ?
I'm not sure this to be logical... And i wonder the reason for this?
Maybe it would be better to remove those lines in index.php (if we should really consider the template option as this one is meant to do) :
// Show the page title here if the header is hidden
if (!$displayHeader) : ?>
<h1 class="content-title"><?php echo JHtml::_('string.truncate', $app->JComponentTitle, 0, false, false); ?></h1>
<?php endif; ?>
Or maybe don't change margin in sidebar, but for the class used when title is in content (around line 294 in template.less file) :
.content-title {
font-size: 24px;
font-weight: normal;
line-height: 26px;
margin-top: 0;
}
By changing the above class, this way, you won't change the sidebar if the header is displayed.
;-)
@JoomliC I believe the result of turning off the header is accurate in its current state. When we hide the header, all its contents: icon/title/logo/styles are no longer visible gaining a small amount of screen real estate and a cleaner appearance(IMHO). We simply are removing the stylization of the title. But the title I see as an important element that needs to remain regardless of what experience you choose.
@ChristopherRaymond Let's go back to the main purpose of your PR : better position of the sidebar, if Isis Template option "Display Header" is set to "No" ;-)
So, don't change the class .j-sidebar-container, this one is good when Header is displayed!
Please, better edit the class .content-title, which one exists for the case the Header is not displayed, but only the title (this what you want!)
For example :
1.
.content-title { font-size: 24px; font-weight: normal; line-height: 26px; margin-top: 0; padding-bottom: 10px; }
Or another one, maybe more clean (in LTR, and with a float left on RTL) :
2.
.content-title { font-size: 24px; font-weight: normal; line-height: 26px; margin: 0 auto 10px; float: right; }
The second one has my preference, because in this case, the sidebar has no changes with or without the Header displayed.
;-)
@ChristopherRaymond You're welcome ;-)
Don't forget to edit template.less, and use the generatecss.php as @zero-24 told you ;-)
And if changes in left/right, don't forget to update too the RTL less file if needed.
In all cases, don't hesitate to ask for help if needed.
When updated, will test and give feedback ;-)
Cyril
@zero-24 @JoomliC - I believe I updated the less file correctly. My apologies in advance if not.
@ChristopherRaymond Sorry for slow response...
In fact, no it's not good, rtl less not updated.
So, i will try to help as far as i can.
The css should be updated only in less files.
Then, use generatecss.php script which will automatically create the template.css and template-rtl.css useing the less files of the template, which are loading too the core less files (icomoon, buttons... and a lot of predefined css for bootstrap or else use by joomla).
Do you develop on local and if yes on windows or mac ? (i know the process on mac using terminal, not on windows, but may be very similar)
link: https://docs.joomla.org/Joomla_LESS
About your changes, i will recommend you to open this PR and change only for the issue with display header, because if you add a styling change there too, this one could take more time before to be commit to staging! (look at the debate here just for one icon change : #6310)
So, in my opinion, focus this PR only on Display header, without changes to sidebar, and your PR will be able to be merge faster.
And if you want to propose a better styling for button, maybe check first my PR here, and open a new one when ready : #6327
This way, could be better and easier first ;-)
In this case, only the padding-bottom :10px added to .content-title in template.less, then use generatecss.php (good to learn how to use it, so that, the next time, you will be ready! ;-) )
This way, will be fast to be tested, the issue with display header option will be fixed!
Cyril
Thanks @JoomliC I will take a look. I am on a MAC. I'll deal with this first then open a new request as suggested regarding the arrow styling.
I'll deal with this first then open a new request as suggested regarding the arrow styling.
@ChristopherRaymond what is the status here? Can we close here if you open a new issue?
Yo van close it.
We have test it a few Times.
Op 1 apr. 2015 18:02 schreef "zero-24" notifications@github.com:
I'll deal with this first then open a new request as suggested regarding
the arrow styling.@ChristopherRaymond https://github.com/ChristopherRaymond what is the
status here? Can we close here if you open a new issue?—
Reply to this email directly or view it on GitHub
#6222 (comment).
Thanks so i'm closing here as requested.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-04-02 10:10:23 |
@test
in joomla 3.4 " Article Manager: Articles" is already visable but this patch corrected the sidebar: now there is space between the sidebar and the toolbar.
i think it is an improvement.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.