? Success
Pull Request for # 5658

User tests: Successful: Unsuccessful:

avatar pe7er
pe7er
12 Jan 2015

If you change the width of the JGLOBAL_HITS from 10% to 1%, the alignment of the Hits column looks better.
Fix for #5658

avatar pe7er pe7er - open - 12 Jan 2015
avatar jissues-bot jissues-bot - change - 12 Jan 2015
Labels Added: ?
avatar brianteeman brianteeman - change - 12 Jan 2015
Rel_Number 5658
Relation Type Pull Request for
avatar brianteeman
brianteeman - comment - 12 Jan 2015

Dont you think it is better to just center align the heading


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

avatar brianteeman
brianteeman - comment - 12 Jan 2015

Scrub that idea

The problem form what I can see is that the header and the data are aligned center

I would have thought that the numbers were easier to read if they were aligned left or right. Its definitely more normal for columns of numerical data to be aligned right than centered

You need to test with data longer than one or two digits eg hits of 12345

avatar pe7er
pe7er - comment - 13 Jan 2015

I tested the Hits field with the number 1,000,000 :-)
Agreed, alignment to the right of Header + hits number in the rows sounds good.

avatar brianteeman
brianteeman - comment - 13 Jan 2015

The comment about the number of hits was for testers ;)
On 13 Jan 2015 00:59, "Peter Martin" notifications@github.com wrote:

I tested the Hits field with the number 1,000,000 :-)
Agreed, alignment to the right of Header + hits number in the rows sounds
good.


Reply to this email directly or view it on GitHub
#5696 (comment).

avatar uthorat
uthorat - comment - 13 Jan 2015

Nice job guys :)


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

avatar uthorat uthorat - test_item - 13 Jan 2015 - Tested successfully
avatar roland-d roland-d - change - 14 Jan 2015
Status Pending Ready to Commit
avatar roland-d
roland-d - comment - 14 Jan 2015

@test: all good. Works as described.


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

avatar roland-d roland-d - change - 14 Jan 2015
Milestone Added:
avatar roland-d roland-d - reference | 98ec4c1 - 14 Jan 15
avatar roland-d roland-d - merge - 14 Jan 2015
avatar roland-d roland-d - close - 14 Jan 2015
avatar roland-d roland-d - close - 14 Jan 2015
avatar roland-d roland-d - change - 14 Jan 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-01-14 12:06:20
avatar pe7er pe7er - head_ref_deleted - 5 Nov 2015

Add a Comment

Login with GitHub to post a comment