Skip to content

Update code element from extending screen width on extra parameters field#409

Open
Saurinpatel20 wants to merge 2 commits intocashapp:masterfrom
Saurinpatel20:master
Open

Update code element from extending screen width on extra parameters field#409
Saurinpatel20 wants to merge 2 commits intocashapp:masterfrom
Saurinpatel20:master

Conversation

@Saurinpatel20
Copy link
Copy Markdown

On larger pieces of text, the code element spans the entire width of the screen and extends the text past the width of the page, which makes buttons on the right side of the screen hidden and hard to find without scrolling all the way over.

Updated the styling on this element so that instead of making it overflow and extend the width of the page, we break the word into separate lines. Used break-all as opposed to break-word.

@Saurinpatel20
Copy link
Copy Markdown
Author

Saurinpatel20 commented Mar 25, 2025

@mpawliszyn Is it possible to get this merged or get feedback on it? It's a quick styling change...

@mpawliszyn
Copy link
Copy Markdown
Collaborator

Right, so note that we will be moving to the new UI soon.
https://github.com/cashapp/backfila/blob/06e2c19a87885ce5916d4af2b06a04d33fc72245/service/src/main/kotlin/app/cash/backfila/ui/components/AlertUseOldUI.kt

Have you run the development app and confirmed this works as you'd expect?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants