User tests: Successful: Unsuccessful:
there is no line between the option to show/hide featured articles and the amount of articles to show in this module.
Without the line people (read: me) can read it as: How many featured articles can be shown. Therefor I was looking for the field to set the amount of other articles I can show.
With the line between the option show/hide featured articles and the field amount people (read: me) have better understanding of the meaning of amount.
Labels |
Added:
?
|
Before
After
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5199.
Personally I didnt have the problem that @hans2103 did ut I see no reason not to apply this change
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5199.
Don't like wasting even more space. Maybe swapping them improves understanding.
@sovainfo
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5199.
OK here too for swapping instead of this.
Dont really care which solution is chosen but as the consensus appears to be for simply just swapping the two fields around and not having the separator line @hans2103 can you update your PR for that and then it can be committed
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5199.
I agree on the suggestion to swap the fields instead of adding an extra element.
therefor I've changed the pull request.
Probably unimportant if this is quickly merged, but the PR title is now out-of-sync with nature of the changes...
Title |
|
I wanted to create a new pull request to remove the obsolete separator line.
It seems that it is merged into this one.
It's ok.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-11-27 05:40:37 |
This refers to mod_articles_category
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5199.