Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow image-only category description #4462

Closed
wants to merge 1 commit into from
Closed

Allow image-only category description #4462

wants to merge 1 commit into from

Conversation

zenexpert
Copy link
Contributor

@scottcwilson
Copy link
Sponsor Contributor

I'm not wild about this. The description is supposed to be a block of text. Could this be a plugin instead?

@zenexpert
Copy link
Contributor Author

It's not a matter of turning it into an additional plugin. Here's a c/p from the forum:

What I'm saying - we either need a warning in messagestack to let the store owner know that the code was stripped down, or we shouldn't strip it down. Or maybe something else, dunno. Just not a silent strip down.

There's no clear info that it SHOULD be a block of text or anything alike - imagine the frustration not knowing what it should be and you add 10-20 images before realizing it's not working and there's no explanation why...

@scottcwilson
Copy link
Sponsor Contributor

This issue has come up exactly once in the 15 years I have been doing this. It's an edge case and a bad idea (as noted by @dbltoe on the forum). There's already a category image if you want to display an image. Descriptions and titles are text.

@proseLA
Copy link
Sponsor Contributor

proseLA commented Jan 6, 2022

i disagree. just because you have only seen it once in 15 years does not mean it's not a problem.

it's another one of those hard coded things that ZC does that has frustrated a valued member of the community. i think the downside of allowing 4 different tags is minimal.

and more importantly, if you strip them and NOT tell the admin they were stripped, i think that's bad communication.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants