#Usability

commented on the other as well: #3485202-30: Change the dialog modal heading from span to h1 with the uiDialogTitleHeadingLevel option added in jQuery UI 1.14.1 →

I guess it happens because Claro is using content property for drawing the separators. The possible fix would be to use background-image instead like Olivero does.

Left a brief comment: #3370326-13: Refine field descriptions →

Left a brief comment: #3370326-13: Refine field descriptions →

Usability review

We discussed this issue at 📌 Drupal Usability Meeting 2024-10-25 Active . The link to the recording of the meeting is https://youtu.be/yuaPVd6Lgv0. For the record, the attendees at the usability meeting were @avani.bhut, @benjifisher, @rkoller, @rosendofig, @shaal, @simohell, and @worldlinemine.

And we have discussed this issue another time at 📌 Drupal Usability Meeting 2025-01-03 Active . That issue will have a link to a recording of the meeting. For the record, the attendees at todays usability meeting were @benjifisher, @rkoller, and @simohell.

In both meetings we’ve continued the wordsmithing of the micro copy for the field type options in the second step of the field creation flow. The work in progress can be found in column C in the spreadsheet (https://docs.google.com/spreadsheets/d/1Lx7L40eRHotr5KQGn6au5WxQO3lFv19a...). We plan to continue reviewing the current draft found in column B/D in the upcoming meetings in case no other issues come up.

not sure why the videos havent been uploaded and added back then. apologies. in both videos i first tab through the breadcrumb and then shift tab back to the start as illustrated by the keycastr output. then i do the same by stepping through the breadcrumb one way pressing VO arrow right... in claro the separators are announced and in focus, in olivero they are ignored.

I'm going to mark this fixed since it has been implemented already, as discussed. Further feedback should go into the core queue for consideration or action.

May need to think approach as not seeing why we are adding comment hooks into node. Should try and keep things separate if possible.

The tests seem fixed now, pipeline runs all green except for the test-only test: fails like it should.

That seems to leave several out-of-scope typehints that need to be removed from the MR.

Left comments.

Rebased

Hi, I was not able to reproduce.

Can you upload the videos that you mensioned?

Appears to need a rebase.

Automatically closed - issue fixed for 2 weeks with no activity.

benjifisher → credited rkoller → .

benjifisher → credited simohell → .

There's a lot of re-work needed to make this work w/ CK5 and D11. https://www.drupal.org/node/3291493 → deprecated the image dialog forms. I'm going to leave this at NW for now. But I started some of the rebase work in https://git.drupalcode.org/project/drupal/-/merge_requests/10767. It is a pure rebase. From here on, we'll need to figure out how to make this work with CK5. I'm not taking that on at the moment.

Added some code review feedback to the MR.

Changed the parent issue to 🌱 [Plan] Improve field creation experience Active ,which is still active, in contrast to ✨ Make field selection less overwhelming by introducing groups Fixed which is closed and fixed. for a while now. otherwise this issue might slip through.

Hi @charles belov, I have thought to rename the display of 'Local tasks' to 'Contextual Tools', raised the MR as well for it.
The reason for selecting this name is that it clearly conveys that the menu provides tools specifically related to the current context.
Please review and share your opinion on it.

This looks good to me, just a couple of comments about the test - one about avoiding random failures we've only just found in the last week or so - and another about refining how we clear the cache

Will keep an eye out for the re-RTBC but feel free to ping me

Thank you for that suggestion. However, as with "Local Tasks," "Quick Links" does not imply it relates to the currently-displayed node.

What about "This Node"?

Although "node" is also jargony.

"This Page" if it's content?
"This User" if it's a user page?

Automatically closed - issue fixed for 2 weeks with no activity.

Replace "Local Tasks" with "Quick Links" to make navigation more intuitive and user-friendly for non-technical users as well. Raised an MR #108

Thank you for posting that it could be just the thing to give inspiration to someone who needs it! Keep up the great work! MarlboroRewards

Wow! Looking forward to it! Thank you all!

Commenting and making notes on content fragments without the field limitations is such an important feature.

Thank you for posting that it could be just the thing to give inspiration to someone who needs it! Keep up the great work! Panda Express Feedback - Welcome

Probably needs usability sign off

MR 10750 is not ready for review, but it is ready for discussion.

The default edit form for the menu item has text like this:

This link is provided by the Standard module. The title and path cannot be edited.

That example is from the Home link in the Main navigation menu. It means the Standard profile: there is no Standard module. The menu_link_content module provides a different form that does not have this text.

As a first step, I added the Provider column to the table of menu links, making that same information available without opening the edit form.

I also added the Expanded column. (The MR needs work: the choices made in this column are ignored when the form is saved.) That is out of scope for this issue, so I should probably make a new issue for it. Or we could expand the scope of this issue: add more information to the table of menu links from the edit forms for those links.

Screenshot using 11.x:

Screenshot using the branch from the MR: