? Success

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
26 Dec 2014

Ok so this is a quick fix and directly addresses the reporters initial concerns (#5524) in the major areas of the first page of the admin (two modules, popular and latest) and the main content list page (Articles Manager). It doesn't intend to be a complete solution but shows how simple it can be to implement.

Votes

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

avatar PhilETaylor PhilETaylor - open - 26 Dec 2014
avatar jissues-bot jissues-bot - change - 26 Dec 2014
Labels Added: ?
avatar PhilETaylor PhilETaylor - test_item - 26 Dec 2014 - Tested successfully
avatar smz
smz - comment - 26 Dec 2014

@PhilETaylor I strongly suspect this has any practical relevance beside making Joomla! more "idiot-proof", but I agree it can be done!

Only thing, you should also modify template.less with the same mod you did in template.css...

avatar brianteeman
brianteeman - comment - 26 Dec 2014

As @smz says you need to edit the less file and then create the CSS


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5525.
avatar PhilETaylor
PhilETaylor - comment - 26 Dec 2014

….and also modify the other admin template…. Im busy testing this :)

On 26 Dec 2014, at 16:03, Brian Teeman notifications@github.com wrote:

As @smz says you need to edit the less file and then create the CSS

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5525.

Reply to this email directly or view it on GitHub.

avatar PhilETaylor
PhilETaylor - comment - 29 Dec 2014

ok this is now in the less files as well - its a quick fix for a minor issue that no one really has - so nothing really important and can be tested and merged :)

avatar brianteeman
brianteeman - comment - 29 Dec 2014

What about hathor? If someone is going to waste their time on this then it might as well be perfect


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5525.
avatar PhilETaylor
PhilETaylor - comment - 29 Dec 2014

er... I did... but....er.... the push doesn't have it for some reason... darn git cherry pick not working so well...

I'll do it again :)

avatar PhilETaylor
PhilETaylor - comment - 31 Dec 2014

Ok I have now managed to sort out the problem with this, caused by a conflict with another PR #5573, so merged that into my branch and resolved the conflicts (easy really) and repushed the missing hathor changes, the PR now looks more complete. Sorry for the delay.

avatar brianteeman
brianteeman - comment - 31 Dec 2014

all good


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5525.
avatar brianteeman brianteeman - test_item - 31 Dec 2014 - Tested successfully
avatar infograf768 infograf768 - close - 12 Jan 2015
avatar infograf768 infograf768 - change - 12 Jan 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-01-12 08:04:39
avatar infograf768
infograf768 - comment - 12 Jan 2015

Corrected a few errors (no need to add the new class in hathor rtl.css as it is loaded after template.css, use tabs instead of spaces), and merged. Thanks.

Add a Comment

Login with GitHub to post a comment