fix: Add white-space CSS to span tag #9626 - #9708
Conversation
|
Welcome to the Appsmith community! Thank you for your first pull request and making this project better. 🤗 |
|
/ok-to-test sha=f5a1f8f |
|
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1569109870. |
|
@Jerimo spaces are recognised now but there seems to a mismatch in amount of space cud u check it out. |
|
@marks0351 ok. I'll work on it. |
|
This PR has not seen activitiy for a while. It will be closed in 7 days unless further activity is detected. |
|
@marks0351 Oh thank you, I didn't spend a lot of time on it these days. I'll try little more and contact you soon. |
|
sorry for the late response @marks0351 . In my opinion, the data displayed in the text widget has the correct number of spaces. Changing the |
marks0351
left a comment
There was a problem hiding this comment.
yea i see that the spaces are accurately shown but the pixels used for each space on the editor vs on the widget seem to different.
|
@Jerimo can u pls pull latest release? |
…ce-in-text-widget
|
Someone is attempting to deploy a commit to the Appsmith Team on Vercel. A member of the Team first needs to authorize it. |
|
@marks0351 i'm done. cud u check it out? |
|
/ok-to-test sha=36a16a1 |
|
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1648156075. |
|
/ok-to-test sha=36a16a1 |
|
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/1649754364. |
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/get-appsmith/appsmith/8HEBAzHDfzodSGnEmwvYCQ9vtWvz |
|
merging his PR because the failing cases are unrelated to the PR |
Co-authored-by: goorm <goorm@RayKim.local>



Description
I fixed an issue where whitespace was not properly reflected in the text widget. I found that the CSS for
white-spacedoesn't exist, and I added it.Fixes #9626
Type of change
How Has This Been Tested?
Checklist: