? ? Success

User tests: Successful: Unsuccessful:

avatar manojLondhe
manojLondhe
9 Jul 2015

:nail_care: This is a PR for fixing alignment issues for list views from com_installer component.

Manage view - before
installer-manage-before
Manage view - after
installer-manage-after

Update view - before
installer-update-before
Update view - after
installer-update-after

Discover view - before
installer-discover-before
Discover view - after
installer-discover-after

Update sites view - before
installer-updatesites
Update sites view - after
installer-updatesites-after

avatar manojLondhe manojLondhe - open - 9 Jul 2015
avatar manojLondhe manojLondhe - change - 9 Jul 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Jul 2015
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 9 Jul 2015

I agree with the movement of the status column
BUT
To be consistent with the other manager views the columns should be left
aligned (I think status and numeric columns are the only ones that are
currently center aligned). It IS a bug that the columns are currently
centered so instead of making the title center aligned please make the
heading left aligned instead

On 9 July 2015 at 10:30, Manoj L notifications@github.com wrote:

[image: :nail_care:] This is a PR for fixing alignment issues for list
views from com_installer component.

Manage view - before
[image: installer-manage-before]
https://cloud.githubusercontent.com/assets/1375006/8592119/65f0cb9a-264a-11e5-8a16-d7012a914b9b.png
Manage view - after
[image: installer-manage-after]
https://cloud.githubusercontent.com/assets/1375006/8592117/65ee0f04-264a-11e5-8ed2-a38e612eb33f.png

Update view - before
[image: installer-update-before]
https://cloud.githubusercontent.com/assets/1375006/8592120/65f17b62-264a-11e5-8a33-016f5bd3aa50.png
Update view - after
[image: installer-update-after]
https://cloud.githubusercontent.com/assets/1375006/8592116/65ed6590-264a-11e5-9785-4b1c263fab46.png

Discover view - before
[image: installer-discover-before]
https://cloud.githubusercontent.com/assets/1375006/8592118/65ef155c-264a-11e5-9b94-12605ddd0ae5.png
Discover view - after
[image: installer-discover-after]
https://cloud.githubusercontent.com/assets/1375006/8592115/65d67e2a-264a-11e5-91bb-b5984aef93e8.png

Update sites view - before
[image: installer-updatesites]
https://cloud.githubusercontent.com/assets/1375006/8592121/65fb60f0-264a-11e5-9970-927f7bd3c00c.png
Update sites view - after
[image: installer-updatesites-after]

https://cloud.githubusercontent.com/assets/1375006/8592122/6611e302-264a-11e5-9eaf-8c23353cccb2.png

You can view, comment on, or merge this pull request online at:

#7384
Commit Summary

  • com_installer: Various cosmetic fixes regarding alignment
  • Merge remote-tracking branch 'upstream/staging' into staging

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#7384.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar manojLondhe
manojLondhe - comment - 9 Jul 2015

There is inconsistency across other manager views as well.

  • Module and plugin manager have - status and ID columns center aligned, rest left aligned
  • Template Manager: Styles has Default, ID, Location, Template columns center aligned
  • Template Manager: Styles have - Location, Version, Date, Author center aligned

Similar story for Language Manger as well!

avatar brianteeman
brianteeman - comment - 9 Jul 2015

Yes and they should be fixed ;)

On 9 July 2015 at 10:47, Manoj L notifications@github.com wrote:

There is inconsistency across other manager views as well.

  • Module and plugin manager have - status and ID columns center aligned, rest left aligned
  • Template Manager: Styles has Default, ID, Location, Template columns center aligned
  • Template Manager: Styles have - Location, Version, Date, Author center aligned

Similar story for Language Manger as well!


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

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar manojLondhe
manojLondhe - comment - 9 Jul 2015

OK, will fix those too keeping only - Status, Default and Numeric/ID columns center aligned!

avatar brianteeman
brianteeman - comment - 9 Jul 2015

See #6076 for some previous discussion on this


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

avatar manojLondhe
manojLondhe - comment - 9 Jul 2015

Sure!
Just noticed com_users using class="left" for table headings, but that class does not exist in backend default template CSS, neither does "right".
Using "right" for numerics makes sense[#6076] but that class does not exist in template, so should I stick with "center"?

avatar brianteeman
brianteeman - comment - 9 Jul 2015

I would go with
#6076 (comment)

On 9 July 2015 at 11:09, Manoj L notifications@github.com wrote:

Sure!
Just noticed com_users using class="left" for table headings, but that
class does not exist in backend default template CSS, neither does "right".
Using "right" for numerics makes sense[#6076
#6076] but that class does
not exist in template, so should I stick with "center"?


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

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar zero-24 zero-24 - change - 9 Jul 2015
Category Language & Strings UI/UX
avatar zero-24 zero-24 - change - 9 Jul 2015
Easy No Yes
avatar manojLondhe
manojLondhe - comment - 9 Jul 2015

OK, but what about "right", "left" classes that don't exists?

avatar brianteeman
brianteeman - comment - 9 Jul 2015

My understanding from the comments in that other thread is that left is the
default for LTR and right is the default for RTL so no class is needed

avatar manojLondhe manojLondhe - change - 9 Jul 2015
Title
PR for com_installer: Various cosmetic fixes regarding alignment
Various cosmetic fixes regarding alignment for Extension, Module, Plugin, Language, Template Managers
avatar manojLondhe
manojLondhe - comment - 9 Jul 2015

Pushed changes for - Extension, Module, Plugin, Language, Template Managers

avatar brianteeman
brianteeman - comment - 10 Jul 2015
avatar brianteeman brianteeman - test_item - 10 Jul 2015 - Tested successfully
avatar brianteeman brianteeman - change - 10 Jul 2015
Category Language & Strings UI/UX Language & Strings Templates (admin) UI/UX
avatar genesisfan
genesisfan - comment - 11 Jul 2015

@test succesfully tested with FF38.0.5. Well done @manojLondhe looks much better.


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

avatar genesisfan genesisfan - test_item - 11 Jul 2015 - Tested successfully
avatar larapetersen
larapetersen - comment - 11 Jul 2015

Tested succesfully.


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

avatar larapetersen larapetersen - test_item - 11 Jul 2015 - Tested successfully
avatar roland-d roland-d - reference | 979336c - 11 Jul 15
avatar roland-d roland-d - merge - 11 Jul 2015
avatar roland-d roland-d - close - 11 Jul 2015
avatar roland-d roland-d - change - 11 Jul 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-07-11 14:58:09
Closed_By roland-d
avatar roland-d roland-d - close - 11 Jul 2015
avatar manojLondhe
manojLondhe - comment - 11 Jul 2015

@genesisfan Thanks!
Thanks guys :)

avatar zero-24 zero-24 - change - 11 Jul 2015
Milestone Added:

Add a Comment

Login with GitHub to post a comment