joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

improving Menus: Edit Item - moving Publishing fields v3 #36918

Open pe7er opened this issue 3 years ago • 14 comments

Pull Request for Issue https://github.com/joomla/joomla-cms/issues/36918 This PR is against 4.1-dev and makes my earlier PR https://github.com/joomla/joomla-cms/pull/36920 (against the 4.0-dev branch) obsolete.

Summary of Changes

This PR moves in the Menus: Edit Item the Start Publishing and Finish Publishing date to the bottom, so that the Access and Language are easier to use (like in Joomla 3.x)

Testing Instructions

Actual result BEFORE applying this Pull Request

On a multilingual website, go to Menus > Main Menu (en-GB) > Create a new menu item. You have to scroll a lot to reach the Access and Language fields:

menu_item

Expected result AFTER applying this Pull Request

After the PR the Access and Language fields are easier to use because you don't have to scroll past the Start Publishing and Finish Publishing fields:

details-v2

Documentation Changes Required

The screens in the documentation will have to be updated to reflect the new position of the Start Publishing and Finish Publishing date at the bottom right of the screen: https://help.joomla.org/proxy?keyref=Help4.x:Menu_Item:_New_Item/en

pe7er avatar Feb 02 '22 15:02 pe7er

For future reference if you are asked to rebase a pr you do not need to close it and make a new one.

image

image

brianteeman avatar Feb 02 '22 15:02 brianteeman

After the PR the Access and Language fields are easier to use because you don't have to scroll past the Start Publishing and Finish Publishing fields:

Instead the Start Publishing and Finish Publishing fields are harder to use as you have to scroll past the access and language fields.

We really should be doing real testing before making changes

brianteeman avatar Feb 02 '22 15:02 brianteeman

I had some issues with git and the Joomla branches on my local system. So I checked out a crispy fresh 4.1-dev branch to redo my small patch. I didn't know the rebase function in Github. Thanks @brianteeman !

pe7er avatar Feb 02 '22 15:02 pe7er

I have tested this item :white_check_mark: successfully on 5e7b2edf003819bdbbc8eea58ea37c4f6ca29ccf


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

drmenzelit avatar Feb 02 '22 17:02 drmenzelit

I have tested this item :white_check_mark: successfully on 5e7b2edf003819bdbbc8eea58ea37c4f6ca29ccf


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

ChristineWk avatar Feb 02 '22 22:02 ChristineWk

I think the position of publish up/down is wrong at all...we should stick with out standard we have in e.g. articles and have it in the correct follow up tabs. (Also for other fields which are everywhere)

bembelimen avatar Feb 03 '22 11:02 bembelimen

@bembelimen are you suggesting an extra tab "Publishing" in the menu item with these options?


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

RickR2H avatar Feb 16 '22 14:02 RickR2H

RTC


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

RickR2H avatar Apr 21 '22 06:04 RickR2H

See the comment from @bembelimen this PR is not thee correct solution

brianteeman avatar Apr 21 '22 07:04 brianteeman

Back to pending. @bembelimen can you decide on this issue?


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

RickR2H avatar Apr 21 '22 07:04 RickR2H

Like I said, it should be streamlined with the other implementation we have in core already

bembelimen avatar Apr 22 '22 00:04 bembelimen

Like I said, it should be streamlined with the other implementation we have in core already

except modules also have this on the first tab as well. So we're not consistent there either

brianteeman avatar Jun 22 '22 15:06 brianteeman

This pull request has automatically rebased to 4.2-dev.

HLeithner avatar Jun 27 '22 13:06 HLeithner

This pull requests has been automatically converted to the PSR-12 coding standard.

joomla-bot avatar Jun 27 '22 21:06 joomla-bot

As @bembelimen wrote, changes in global parts of our edit screen are against a11y and UX.

chmst avatar Oct 22 '22 14:10 chmst