The color module preview image keeps being misinterpreted as a live preview of the user's actual site with their content, visual changes, logos, etc.
There is this existing issue, where many people have discussed it from a few years back up to right now... #1167366: Replace "Lorem Ipsum" Color preview HTML templates with example text in Twig templates .
And a new issue raised by someone this week... #2396983: Header Logo with Bartik won't change in settings preview.
People keep expecting this image of Bartik to show their actual site which it can't do and shouldn't try to do.
The real purpose of this image is to demo colours picked in the color scheme above it. It is indeed within the Color Scheme fieldset, but people don't seem to immediately understand this.
As an alternative solution to changing the content within the "preview" of Bartik, which is still under discussion here, I think a quicker win and a good start to this in the meantime would be to rename the title above the preview image.
This snapshot of Bartik is not a preview, the real site does not look like this, it's just an example of Bartik. As said in this issue here by John Albin.
I suggest changing the title to "Example" as suggested here or more specifically with no room for error "Example Color Scheme" to define what the image is actually portraying.
Comment | File | Size | Author |
---|---|---|---|
#26 | after-patch.png | 36.75 KB | gaurav-mathur |
#13 | wording_colorscheme_example-2400277-13.patch | 483 bytes | SteffenR |
#6 | wording_colorscheme_example-2400277-6.patch | 483 bytes | gbisht |
#2 | wording_colorscheme_example-2400277-2.patch | 483 bytes | SteffenR |
Bartik___Site-Install.png | 479.65 KB | emma.maria |
Comments
Comment #1
SteffenRComment #2
SteffenRAttached a patch fixing the wording. I choosed "Example Color Scheme" for clarification of content shown below the headline.
As i was thinking about the issue some time the wording "Example Color Scheme" seems to be a bit unspecific in this case - cause the content below shows the color scheme with the actual selected colors - from my understandings an example color scheme would be "Bartik in its default colors".
Maybe we should rename the section to Preview of color scheme - in this case it's clear to the user, that we don't show a preview of the page.
What do you think?
SteffenR
Comment #3
oadaeh CreditAttribution: oadaeh commentedI think "Example" can be misconstrued in a similar way that "Preview" can be. To me, "Demonstration" would be a better choice, and probably better yet, something more like "Demonstration of chosen color scheme".
Comment #4
SteffenR@oadaeh: good point - what do you think @emma.maria
Comment #5
sivaramakrishnan CreditAttribution: sivaramakrishnan commentedNeed to desired on correct text for Preview to Demonstration or Example Color Scheme.
Comment #6
gbisht CreditAttribution: gbisht commentedI think "Preview Color Scheme" is better text here. Attached the patch as well.
Comment #7
SteffenRThanks gulab.bisht - i think we can set the patch / the issue tp RTBC..
Comment #8
emma.mariaI have some more thoughts on this before a committer takes a look at it.
Apologies for losing track of this issue in the pile and not giving feedback earlier.
Comment #9
meramo CreditAttribution: meramo commentedThe patch in #6 still applies against the latest HEAD
Comment #10
aburke626The patch in https://www.drupal.org/node/2400277#comment-9526805 applies cleanly and the label text is changed as expected. I think that "Preview Color Scheme" is a straightforward explanation for what this functionality does.
Comment #11
emma.mariaComment #12
Bojhan CreditAttribution: Bojhan commentedI am pretty sure we don't camel case like that.
Comment #13
SteffenRI hope we can merge the patch into core now - changed the camel casing to "Preview color scheme" in the attached patch.
Comment #25
quietone CreditAttribution: quietone at PreviousNext commentedColor has been removed from core, #3270899: Remove Color module from core.
Comment #26
gaurav-mathur CreditAttribution: gaurav-mathur at Dotsquares Ltd. commentedPatch #13 tested and applied successfully. Attaching screenshot for reference.