Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4.0] Change Status dropdown consistency #24078

Merged
merged 3 commits into from
Mar 4, 2019

Conversation

brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Mar 3, 2019

Having a consistent UI is essential. All off the new change status dropdown items are ordered Published->Unpublished->Archive->Checkin->Trashed except for com_content where Archive and Checkin are reversed.

This PR changes the order of the articles change status dropdown items to be consistent with all the others

Having a consistent UI is essential. All off the new change status dropdown items are ordrered Published->Unpublished->Archive->Checkin->Trashed except for com_content where Archive and Checkin are reversed.

This PR changes the order of the articles change status dropdown items to be consistent with all the others
@Quy
Copy link
Contributor

Quy commented Mar 4, 2019

I have tested this item ✅ successfully on 5170932


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

@Quy
Copy link
Contributor

Quy commented Mar 4, 2019

Please do the same with Featured Articles.

@brianteeman
Copy link
Contributor Author

Updated for featured aarticles

@wilsonge wilsonge merged commit 05622f2 into joomla:4.0-dev Mar 4, 2019
@wilsonge
Copy link
Contributor

wilsonge commented Mar 4, 2019

Thanks!

@wilsonge wilsonge added this to the Joomla 4.0 milestone Mar 4, 2019
@brianteeman
Copy link
Contributor Author

Thanks

@brianteeman brianteeman deleted the change_status branch March 4, 2019 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants