search

zesty-io / manager-ui

Content platform powering enterprise websites and applications
https://www.zesty.io
Other
37 stars 5 forks source link

Content: WYSIWYG character count only includes text #2523

Open shrunyan opened 5 months ago

shrunyan commented 5 months ago

Describe the bug The current character count shown for the WYSIWYG editor only takes into consideration the text string literal. This is inaccurate. As the data stored to the database is the underlying HTML generated by the WYSIWYG editor. This underlying HTML is also included in the limit of what that database field allows. As such it should be inlcuded as part of the character count.

Expected behavior TL:DR; Count all the HTML, not just the user typed text.

Screenshots image

TEST = 4 characters

image

<p>TEST</p> = 11 characters

Additional context @zcolah this probably needs design consideration. As the HTML characters which would impact this character count are not visible to users. As such there maybe confusion on why that character count is different than the character count of the text they inserted.

finnar-bin commented 1 month ago

@zcolah need your confirmation if we're fine with showing the actual character count that's saved to the database compared to what the user sees when typing in the wysiwyg as pointed out by @shrunyan or is there any other design changes to be added unto this task?

zcolah commented 1 month ago

Currently this project is added to our development queue. No action needs to be taken. Not a priority.