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

Article edit tabs and fields

Open chmst opened this issue 1 year ago • 3 comments

Pull Request for Issue #44118 .

Summary of Changes

This PR re-arranges fields and tabs in frontend edit for articles. It moves all options which are in the right sidbar of articles to a new tab "options" It writes all fields which are in the publishing tab of the article into the publishing tab. This removes the language tab and the metadata tab.

Testing Instructions

Compare the edit form for an article before and after the PR. Make sure that different user groups witrh different permissions have the same fields before and after the pr. Swith on and off the show_publishing_data in articles options. on: publishing tab exists, off: no publishing data.

Actual result BEFORE applying this Pull Request

see #44118

Expected result AFTER applying this Pull Request

grafik grafik

REMARK It would be correct to use a new param for switching on and off the publishing tab. But this would be a b/c break.

Link to documentations

Please select:

  • [x] Documentation link for docs.joomla.org:

  • [ ] No documentation changes for docs.joomla.org needed

  • [ ] Pull Request link for manual.joomla.org:

  • [ ] No documentation changes for manual.joomla.org needed

chmst avatar Oct 14 '24 08:10 chmst

I have tested this item :white_check_mark: successfully on 7e079b93076f5f10e51a8ee7036b0aa416410122


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

ghost avatar Oct 17 '24 08:10 ghost

@chmst Can the ordering of fields be the same between Back- and Frontend:

Tab "Options"

Backend Frontend
options-backend options-frontend

Tab "Publishing"

Backend Frontend
publishing-backend publishing-frontend

ghost avatar Oct 19 '24 08:10 ghost

Changed the ordering of fields as suggested by @fgsw , added a fieldset and legend to metadata (ux, a11y)

chmst avatar Oct 19 '24 11:10 chmst

While test i set Article Options in the Backend to "Hide" and was confused as the Tab is shown in the Frontend:

Backend Frontend
be fe

I realised Article Options hide only in the Backend.

To avoid Confusion i suggest to rename in the Frontend the Tab Options to "Details".

Comment: I prefer inline help that a field is for Frontend, Backend or both but thats not in scope of this Pull Request.

ghost avatar Oct 24 '24 08:10 ghost

It is confusing. Please use the Help button grafik

The whole usage opf param is wrong and the contrary of self-explaining. But no chance to change this in Version 5.xdue to b/c

chmst avatar Oct 24 '24 08:10 chmst

I've given this a feature label for now, since I don't think this is a bugfix, correct? In that case it would have to go into 5.3

Hackwar avatar Nov 23 '24 14:11 Hackwar

I've decided that my feature label is correct and thus rebased this to 5.3-dev.

Hackwar avatar Dec 12 '24 11:12 Hackwar

I have tested this item :white_check_mark: successfully on 735b8f5d672180e7a88abd224501b4df1f17cc7d

Tested successfully. Previously, the article edit tabs and fields felt confusing and inconsistent. With this update, the UX is much clearer and more intuitive. A great improvement—thanks! ✅


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

shamsbd71 avatar Feb 22 '25 05:02 shamsbd71

I have tested this issue tested successfully


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

chaitali-tekdi avatar Feb 22 '25 07:02 chaitali-tekdi

@chaitali-tekdi Please submit your test result at the Issue tracker.

ghost avatar Feb 22 '25 07:02 ghost

I have tested this item :white_check_mark: successfully on 735b8f5d672180e7a88abd224501b4df1f17cc7d

Does what it says in the instructions, apart from removing Language and Metadata tabs because they weren't there in the first place.


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

crommie avatar Feb 22 '25 08:02 crommie

I have tested this item :white_check_mark: successfully on 735b8f5d672180e7a88abd224501b4df1f17cc7d


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

shrutidhole123 avatar Feb 22 '25 08:02 shrutidhole123

I have tested this item :white_check_mark: successfully on 735b8f5d672180e7a88abd224501b4df1f17cc7d

I have tested this issue tested successfully


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

chaitali-tekdi avatar Feb 22 '25 08:02 chaitali-tekdi

RTC


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

alikon avatar Feb 22 '25 11:02 alikon

Due to conflicts: Closing this. Will make a new PR for the 6.0 branch.

chmst avatar Mar 05 '25 14:03 chmst