? Success

User tests: Successful: Unsuccessful:

avatar ChristopherRaymond
ChristopherRaymond
27 Feb 2015

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

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar ChristopherRaymond ChristopherRaymond - open - 27 Feb 2015
avatar joomla-cms-bot joomla-cms-bot - change - 27 Feb 2015
Labels Added: ?
avatar ChristopherRaymond ChristopherRaymond - change - 27 Feb 2015
Title
Styling changes to sidebar
ISIS template styling changes to sidebar
avatar ChristopherRaymond ChristopherRaymond - change - 27 Feb 2015
Title
ISIS template styling changes to sidebar
isis template styling changes to sidebar
avatar brianteeman brianteeman - change - 27 Feb 2015
Category Templates (admin)
avatar recreatorshop
recreatorshop - comment - 28 Feb 2015

@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.
avatar recreatorshop recreatorshop - test_item - 28 Feb 2015 - Tested successfully
avatar pe7er
pe7er - comment - 28 Feb 2015

@test

This patch adds some space between toolbar & sidebar.
Before:
screen shot 2015-02-28 at 07 58 01

After installing Patch
screen shot 2015-02-28 at 07 58 01


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.
avatar pe7er
pe7er - comment - 28 Feb 2015

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.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.
avatar pe7er pe7er - test_item - 28 Feb 2015 - Tested successfully
avatar fvdn
fvdn - comment - 28 Feb 2015

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


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.
avatar fvdn fvdn - test_item - 28 Feb 2015 - Tested successfully
avatar josien
josien - comment - 28 Feb 2015

@test
Patch also adds a little styling to the arrow:
screen shot 2015-02-28 at 08 11 21screen shot 2015-02-28 at 08 11 29


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.
avatar ChristopherRaymond
ChristopherRaymond - comment - 28 Feb 2015

@josien Correct. This small style addition adds a visual anchor to this element.

avatar zero-24 zero-24 - change - 28 Feb 2015
Status Pending Ready to Commit
avatar zero-24 zero-24 - alter_testresult - 28 Feb 2015 - josien: Tested successfully
avatar zero-24
zero-24 - comment - 28 Feb 2015

RTC based one various testing. Thanks for testing and coding!


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.
avatar losedk
losedk - comment - 28 Feb 2015

Shouldn't the less file be updated as well? And what about the rtl version?

avatar zero-24 zero-24 - change - 28 Feb 2015
Status Ready to Commit Pending
avatar zero-24
zero-24 - comment - 28 Feb 2015

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


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.
avatar JoomliC
JoomliC - comment - 1 Mar 2015

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; ?>
avatar JoomliC
JoomliC - comment - 1 Mar 2015

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.

;-)

avatar ChristopherRaymond
ChristopherRaymond - comment - 1 Mar 2015

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

avatar JoomliC
JoomliC - comment - 2 Mar 2015

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

avatar JoomliC JoomliC - test_item - 2 Mar 2015 - Tested unsuccessfully
avatar JoomliC JoomliC - test_item - 2 Mar 2015 - Tested successfully
avatar JoomliC JoomliC - test_item - 2 Mar 2015 - Tested unsuccessfully
avatar ChristopherRaymond
ChristopherRaymond - comment - 2 Mar 2015

@JoomliC Good thinking, will make that edit.

avatar JoomliC
JoomliC - comment - 2 Mar 2015

@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

avatar ChristopherRaymond
ChristopherRaymond - comment - 2 Mar 2015

@zero-24 @JoomliC - I believe I updated the less file correctly. My apologies in advance if not.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.
avatar JoomliC
JoomliC - comment - 6 Mar 2015

@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

avatar ChristopherRaymond
ChristopherRaymond - comment - 6 Mar 2015

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.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/6222.
avatar zero-24
zero-24 - comment - 1 Apr 2015

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?

avatar recreatorshop
recreatorshop - comment - 2 Apr 2015

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

avatar zero-24
zero-24 - comment - 2 Apr 2015

Thanks so i'm closing here as requested.

avatar zero-24 zero-24 - change - 2 Apr 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-04-02 10:10:23
avatar zero-24 zero-24 - close - 2 Apr 2015
avatar zero-24 zero-24 - close - 2 Apr 2015
avatar ChristopherRaymond
ChristopherRaymond - comment - 2 Apr 2015

sorry for the late response @zero-24 - I got hung up with the less conversion. Should I continue with this or leave it closed?

Add a Comment

Login with GitHub to post a comment