Skip to content

Conversation

@byseif21
Copy link
Contributor

@byseif21 byseif21 commented Jan 9, 2026

  • fix some rare cases in result for some texts/letters wrap in some screen sizes during screenshot capture, by forcing not to white-space: nowrap;

that happens for word history in custom & zen where ligatures are always on.
for "type mode" word (top class in group testType) & "input history" word (resultWordsHistory's titleclass)

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Jan 9, 2026
&.withLigatures {
.word {
overflow-wrap: anywhere;
//overflow-wrap: anywhere;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

breaks words at arbitrary letters,
I don’t see a valid use for internal word breaking here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if the word gets longer than the words wrapper width?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok i see in custom it's not limited hmm

Comment on lines +765 to +771

// for "input history" wrapping in screenshot for small screens
flex-wrap: wrap;

> span {
white-space: nowrap;
}
Copy link
Contributor Author

@byseif21 byseif21 Jan 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this to fix a prod bug where in small screen bottom class breaks out wrapper class cuz of title and makes the words history goes out "visual overflow" so we let it wrap and span "input history" word not to fix our ss bug

@byseif21 byseif21 marked this pull request as ready for review January 9, 2026 22:28
@github-actions github-actions bot added the waiting for review Pull requests that require a review before continuing label Jan 9, 2026
@byseif21 byseif21 changed the title fix(result-page): some unwanted texts wrapping in result screenshot (@byseif21) fix(screenshot): some unwanted texts wrapping (@byseif21) Jan 9, 2026
@Miodec Miodec added waiting for update Pull requests or issues that require changes/comments before continuing and removed waiting for review Pull requests that require a review before continuing labels Jan 10, 2026
@github-actions github-actions bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label Jan 10, 2026
@byseif21 byseif21 marked this pull request as draft January 10, 2026 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants