search

enonic / xp

Enonic XP
https://enonic.com
GNU General Public License v3.0
202 stars 34 forks source link

Edit-button for each content in the Publishing Wizard #6184

Closed Bellfalasch closed 6 years ago

Bellfalasch commented 6 years ago

Improvement suggested by TINE editors during training.

When publishing any content, any related contents will also be included. It would be extremely handy to have a edit-link on each of these related contents so that I can open it up for editing in a new tab before finally clicking publish. It would improve the workflow around publishing (today you need to close the wizard, navigate to these items in the grid or in the form, and then edit them).

Today it looks like this. To be consistent with UI a pen-icon would be added next to the "x"-icon on the far right side.

screen shot 2018-04-19 at 13 01 42
alansemenov commented 6 years ago

@pmi We already support opening content from that dialog but only for invalid items. Let's always open content wizard on click (there's not much space there for another icon). Also, get rid of "Edit invalid content" tooltip that we show for invalid items - always show tooltip with full path (see https://github.com/enonic/xp/issues/6185)

Do the same for the "main" content above the dependency list - both link and tooltip.

alansemenov commented 6 years ago

Issue moved to enonic/app-contentstudio #49 via ZenHub

sigdestad commented 6 years ago

But how will we visualize that you can click on the content to edit? We don't have this concept anywhere except grid? How about just supporting doubleclick then?

alansemenov commented 6 years ago

We change the mouse cursor to "hand". It's very obvious.

sigdestad commented 6 years ago

Also on a touch device?

alansemenov commented 6 years ago

On touch we could change text color to blue. We really don't have space for another icon there. And having edit and remove icons next to each will be challenging, especially on mobile devices.

sigdestad commented 6 years ago

Color blue? That indicates it is selected? Should all items be blue?

Suggestion: Lets try using "click anywhere except X" == edit, test it out.