User tests: Successful: Unsuccessful:
Pull Request for Issue #16000 .
Second of 2 PRs to apply #16000. Moves the toolbar apply actions to the table header in com_content->articles.
Apply patch and navigate to Content -> Articles. Ensure each item in the dropdown functions correctly.
Yes
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_content Language & Strings Templates (admin) Libraries |
I can move batch back to the toolbar until such time UX can review the best location?
Yes please
On 17 May 2017 3:03 p.m., "Ciaran Walsh" notifications@github.com wrote:
I can move batch back to the toolbar until such time UX can review the
best location?—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#16076 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8d0TMUyz4NObRYhprlA4iBXkwEf4ks5r6v4ZgaJpZM4NdynO
.
Labels |
Added:
?
?
|
Done
Category | Administration com_content Language & Strings Templates (admin) Libraries | ⇒ | Administration com_content Language & Strings Templates (admin) Libraries Front End Templates (site) |
Thank you @brianteeman
Webkit appearance fix committed, resolving this issue.
I have tested this item
Thanks @ciar4n
Click on "Apply Patch" got:
Error
The file marked for modification does not exist: administrator/components/com_content/views/articles/tmpl/default.php
Not sure about this. Having looked at it again now, I don't think a dropdown in the table header would be the best place. The toolbar is where list actions have always resided in Joomla, so I think it would be best to keep them there, and group them.
Submitting that sort of PR would of course get grilled by some just for the sake of it.
@franz-wohlkoenig - that was to do with another PR that was submitted yesterday. Just needs to fix conflicts.
@coolcat-creations Conflicts fixed
Category | Administration com_content Language & Strings Templates (admin) Libraries Front End Templates (site) | ⇒ | Administration com_content Language & Strings Templates (admin) Libraries |
Milestone |
Added: |
Milestone |
Added: |
What is the status on this or is it redundant with the new admin ui
?
Personally I still consider this a viable UI solution. It is not part of the new design.
Can you fix the conflicts here and make the dog happy?
Can you still unpublish multiple articles with this pr?
https://invis.io/2XESGMRA9#/267252691_Content_-_Manage_Articles_-_Item_1_Ticked_-_Actions_Open
you should of course be able to unpublish
I mean multiple items.
Yes thats the indent of the Action Button.
Now I got it right, the actions menu in the table header is for all articles and the three dots are per article. Thanks for clarification.
I applied the current designapproach which makes it probably more obvious :) The concept in the header is not how it will be...
Category | Administration com_content Language & Strings Templates (admin) Libraries | ⇒ | Administration com_content Language & Strings Templates (admin) Libraries External Library Composer Change Front End Templates (site) |
Labels |
Added:
?
|
Conflicts and hound errors fixed
Had a look on it now. I do miss here the connection to the checkboxes, something feels for me not clear, especially when a first time user opens that list. But eager to hear other opinions.
Had a look on it now. I do miss here the connection to the checkboxes, something feels for me not clear, especially when a first time user opens that list. But eager to hear other opinions.
We could hide the 'actions' button until an item has been selected. Animating the button in on selection I believe would make more of a visible connection between the selected items and the button. It is hard for me to look at this with 'new' eyes to completely agree with you.
Regardless maybe best putting this on hold until the template has changed. It is only within the new design that the point can be properly tested. It would make for a good userbob test.
If you click through the draft, the button was designed "inactive" until the user selects something. This is due to usability. Just start from the beginning and click yourself through to the article overview: https://projects.invisionapp.com/share/8SES9JLZB#/screens/266861164_Desktop_-_Login
This should be closed as the concept has been applied in another pr. My 2c
Closed as suggested.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-03-29 18:40:54 |
Closed_By | ⇒ | ciar4n |
personally i wouldnt move batch to this group but thats something for a ux team review