Hide View Mode behind a feature toggle

Created on 24 March 2023, over 1 year ago
Updated 9 June 2023, over 1 year ago

Problem/Motivation

An interesting detail from the design review / user survey is that they don't want advanced features in the preview. They want just the preview.

They specifically don't know what a view mode is and don't need the feature. Maybe developers do, so we should keep the feature, but we should hide it by default and only enable it if the user opts-in.

Steps to reproduce

Proposed resolution

Wrap the display of the view mode behind a setting.

Remaining tasks

User interface changes

API changes

Data model changes

🌱 Plan
Status

Fixed

Version

1.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States cosmicdreams Minneapolis/St. Paul

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @cosmicdreams
  • @cosmicdreams opened merge request.
  • Status changed to Needs review over 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States cosmicdreams Minneapolis/St. Paul
  • πŸ‡ΊπŸ‡ΈUnited States cosmicdreams Minneapolis/St. Paul

    screenshot of this working. Here I'm logged in as the "author" role

  • πŸ‡ͺπŸ‡ΈSpain ckrina Barcelona

    I've tried this and I can't seem to find the place to enable/disable this. I assume the settings should have to show up in /admin/config/system/same-page-preview, but I don't see them as admin role.

    Also, it would be useful to have the config page be linked from the Extend page.

  • πŸ‡ΊπŸ‡ΈUnited States cosmicdreams Minneapolis/St. Paul

    Ah, good point. This particular config is on the content type's "Edit" page. The config page we added has been kept around a while thinking that it could be a good place to put a feature toggle. But everytime we look to provide an option like that we discover that the config can be placed somewhere else.

    I'll open a ticket to just remove that config page, and link to 2 places where the permission gates are managed: User permissions and Content Type settings.

  • πŸ‡ΊπŸ‡ΈUnited States cosmicdreams Minneapolis/St. Paul

    Second thought: I'll just do that here.

  • Status changed to Fixed over 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States brianperry

    Marking as fixed as we now have an option to disable this under the settings button in 2.1.x

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

Production build 0.71.5 2024