User tests: Successful: Unsuccessful:
This may be personal preference but I found the background on the "add module to dashboard" too "in your face".
With this PR you can now see the background pattern etc which I believe to be more accurate representation of the original design instead of the solid gray.
Simplified the PR to just replace the solid gray colour with the patterned background
dont forget to run rnom i or node build.js --compile-css
Status | New | ⇒ | Pending |
Category | ⇒ | Administration Templates (admin) |
I agree with the subtle approach versus the harsh white.
I disagree with same color as background. Ok, the white is really too bright. But there should be some change beside the "+" disappearing when you hover it so you can see something is activated which you can click or tip on. Maybe a grey lighter than the background would be good?
@richard67
++
I am not fun of both: not original, and not suggested.
tbh, the big plus icon that crosses with text looks to me as a bug. Would be cool to find a better design idea for it.
Yes, original is also not nice. But I have no idea right now what would be optimal.
I definitely would not put it there as it would not be logical
I definitely would not put it there as it would not be logical
I would prefer another position too
The suggestion here maintains the objectives of the original design
Only my opinion, it just looks unfinished if you always have a add more area that looks like a module.
I believe that was the designers intention
@HLeithner
Your proposal would not work as we can unpublish some dashboard modules.
The button at bottom is good and intuitive, except its styling
bs convention is for lighten/darken to be 10%
I don't say that my solution is good ;-)
I only hate this big Add button that looks like a module and I can't remove it.
Giving this to a customer looks like I forgot something. And to be honest the use case for this buttons is more or less after installation and configuration afterwards you it's really unlikely that you add another module.
Also if you create the perfect dashboard and want to use the full space you always have a scrollbar...
But I don't want to discuss this any longer, it is what it is.
Do we need the + icon? In general we have a + OR the word "Add ..." not both.
If there is no +, then a 10% lightening would be sufficient and is not so "in your face".
That was the design the designer created and tbh facebook people seem to like useless wizzbang stuff like this.
Removing it is not part of this PR. I was just fixing the background to be closer to the original design
Labels |
Added:
?
|
simplified the pr so that it is super simple and an obvious merge
Title |
|
I have tested this item
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-02-05 17:46:44 |
Closed_By | ⇒ | rdeutz | |
Labels |
Added:
?
|
Thanks all
I have tested this item✅ successfully on 70d06e4
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27468.