? Success

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
17 Oct 2014

Consistency

As stated here #4694 (comment) when editing a message the main menu is still available. This differs from the rest of the admin area behavior. This PR unifies it.

test

Apply patch
go to /index.php?option=com_messages&view=message&layout=edit
Main menu should be disabled.

avatar dgt41 dgt41 - open - 17 Oct 2014
avatar jissues-bot jissues-bot - change - 17 Oct 2014
Labels Added: ?
avatar allanlhansen allanlhansen - test_item - 17 Oct 2014 - Tested successfully
avatar brianteeman brianteeman - change - 17 Oct 2014
Category UI/UX
avatar losedk
losedk - comment - 17 Oct 2014

All good!

avatar losedk losedk - test_item - 17 Oct 2014 - Tested successfully
avatar jcata jcata - test_item - 17 Oct 2014 - Tested successfully
avatar 1apweb 1apweb - test_item - 17 Oct 2014 - Tested successfully
avatar betweenbrain
betweenbrain - comment - 17 Oct 2014

This change makes sense to me. I'll test the PR and then merge it if all goes well.

avatar betweenbrain betweenbrain - close - 17 Oct 2014
avatar betweenbrain betweenbrain - change - 17 Oct 2014
Title
Hide menu when editing a message
Disable menu when editing a message
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-10-17 16:35:21
avatar betweenbrain
betweenbrain - comment - 17 Oct 2014

Merged via 7b78718. Thanks @dgt41!

Add a Comment

Login with GitHub to post a comment