User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Improve UX. Users can select a number of columns from a dropdown. (was a text input field).
Compare menu items for blog and featured before and after Patch.
Status | New | ⇒ | Pending |
Category | ⇒ | Front End com_content |
Not sure about this being a dropdown as it enforces a maximum number of columns. Why not just correct the type of the field from text to number.
I have tested this item
@brianteeman The css is made for 4 columns maximum.
for cassiopeia but what about a custom template?
Touchée. Will commit asap.
I have tested this item
I think, 4 columns are enough :-)
I think, 4 columns are enough :-)
Really depends on the use case.
@chmst welcome -> I am just talking and you have the work
@brianteeman @ChristineWk
4 columns are enough-> if we can be flexible here, it would be great. If not we have to deal width 4
maybe an example : If we want to build a horizontal snapslider it would be great to select 10 columns
Labels |
Added:
?
|
Category | Front End com_content | ⇒ | Administration com_content Front End |
Thank you all. I have changed the dropdowns to number field.
I think, 4 columns are enough :-)
Really depends on the use case.
yes of course :-)
you are too fast :-) following just for info:
4 columns not possible if there are modules on the right position :-)
amendments to number fields works now, but a question:
The arrows are inactive?
4 columns not possible if there are modules on the right position :-)
depends on the template .-)
I have tested this item
The arrows are inactive?
@ChristineWk i think they show that its a number field, arrow-up -down works. But first i tried to click with the mouse :-)
I have tested this item
4 columns not possible if there are modules on the right position :-)
depends on the template .-)
of course :-) @angieradtke
I have tested this item
Title |
|
I've allowed myself to change the title of this PR so it fits to latest changes for using number fields.
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
?
|
I've updated the branch to latest 4.0-dev to get rid of the unrelated javascript-cs error in drone and restored the previous test results in the issue tracker.
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-05-02 19:52:07 |
Closed_By | ⇒ | drmenzelit | |
Labels |
Added:
?
Removed: ? |
Thanks
Thanks for the nudge @angieradtke