The Views UI module provides a tour in config/install/tour.tour.views-ui.yml.

This tour has a lame label (which is supposed to be human-readable) of "Views ui".

It needs a better label. I was trying to make a list of tours for #2374083: Add tours list to admin/help page and this doesn't describe what the tour is at all, plus "ui" isn't even correct (it would be "UI").

Beta phase evaluation

-->

Reference: https://www.drupal.org/core/beta-changes
Issue category Bug because this human-readable text isn't human-readable.
Unfrozen changes Unfrozen because it only changes one translatable UI text string
Prioritized changes Prioritized as it is usability/UI and a bug fix.
CommentFileSizeAuthor
#1 2386005.patch451 bytesjhodgdon

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Issue summary: View changes
Status: Active » Needs review
Issue tags: +Quick fix
StatusFileSize
new 2386005.patch451 bytes

Here's a proposed patch that changes the human-readable label to "View edit page", because it's a tour that gets displayed when you are editing a view.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Needs review » Reviewed & tested by the community
dawehner’s picture

dawehner
Primary language German
commented 
--- a/core/modules/views_ui/config/install/tour.tour.views-ui.yml
+++ b/core/modules/views_ui/config/install/tour.tour.views-ui.yml

+++ b/core/modules/views_ui/config/install/tour.tour.views-ui.yml
@@ -1,6 +1,6 @@

@@ -1,6 +1,6 @@
 id: views-ui

Did you considered to rename the machine name as well ... the views UI also contains the list of all views but yeah it does not matter that much as the label

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

There is currently only one tour in the Views UI module. Probably the Views list page doesn't need a Tour... so I think it is OK to leave it as-is.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Fair enough. Committed d4e03f2 and pushed to 8.0.x. Thanks!

Thanks for adding the beta evaluation for to the issue summary.

  • alexpott committed d4e03f2 on 8.0.x 
    Issue #2386005 by jhodgdon: Views UI tour needs better label

Status: Fixed » Closed (fixed)

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