Problem/Motivation

A picture is worth a thousand words

So adding screenshots to the module page would be super helpful to easily show what the module does.

Furthermore, it still has Drupal 7 information which is EOL.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#11 view_mode_previews.png63.69 KBsamlewis
#7 view-mode-rss.png73.7 KBpallabi_mondal
#7 view-mode-eck.png170.27 KBpallabi_mondal
#7 view-mode-default.png622.27 KBpallabi_mondal
#7 view-mode-previews.png124.62 KBpallabi_mondal
#7 preview-display-tab.png346.66 KBpallabi_mondal

Issue fork view_modes_display-3587171

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

anybody created an issue. See original summary.

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented

I've removed D7 info

pallabi_mondal made their first commit to this issue’s fork.

pallabi_mondal’s picture

pallabi_mondal commented

Added screenshots to the module page and updated the README accordingly.

pallabi_mondal’s picture

pallabi_mondal commented
Status: Active » Needs review
pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Needs work

> Added screenshots to the module page

how? :)

I think you need to create the MR, not just push to the branch, also the images shouldn't be in the readme, you can post them as attachments here so we don't make the repo bigger than it should.

I would recommend the screenshots to include only the relevant bit you want to show rather than the whole screen.

pallabi_mondal’s picture

pallabi_mondal commented
StatusFileSize
new preview-display-tab.png346.66 KB
new view-mode-previews.png124.62 KB
new view-mode-default.png622.27 KB
new view-mode-eck.png170.27 KB
new view-mode-rss.png73.7 KB

Removed the screenshots from the README as suggested and attached updated screenshots here to demonstrate the module in use.

pallabi_mondal’s picture

pallabi_mondal commented
Status: Needs work » Needs review
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

Thanks, I think https://www.drupal.org/files/issues/2026-06-22/view-mode-previews_0.png is fine.

D7 info has already been removed, as it looks?

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs review » Needs work

Agreed, could we have a screenshot of https://www.drupal.org/files/issues/2026-06-22/view-mode-previews_0.png without the warning? I think that would be enough :)

Also, if you're changing the README in a Merge Request/branch, please click on the "Create merge request" link in gitlab

samlewis’s picture

samlewis commented
StatusFileSize
new view_mode_previews.png63.69 KB

Hi, I have uploaded the View Mode Preview screen without the warning.
Let me know if anything else is needed, thanks

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Needs work » Fixed

I think we're done here? :)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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