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

Wrapping pre quotes for readability #9

Open
mcint opened this issue Nov 7, 2022 · 2 comments
Open

Wrapping pre quotes for readability #9

mcint opened this issue Nov 7, 2022 · 2 comments

Comments

@mcint
Copy link

mcint commented Nov 7, 2022

On recent discussion, https://hackerweb.app/#/item/33512338
contrast the readability of the native hackernews quote
Screen Shot 2022-11-07 at 14 37 49

with the readability of the hackerweb quote.
Screen Shot 2022-11-07 at 14 37 17

mcint added a commit to mcint/hackerweb-native-2 that referenced this issue Nov 7, 2022
@mcint
Copy link
Author

mcint commented Nov 7, 2022

If this change is correct, we should see pre-blocks Screen Shot 2022-11-07 at 14 58 21

Choice of style

pre-wrap maintains the pre-text and other whitespace to wrap where needed.

MDN has an example to play with, as does the post linked in this issue.

mcint added a commit to mcint/hackerweb-native-2 that referenced this issue Nov 7, 2022
@cheeaun
Copy link
Owner

cheeaun commented Nov 8, 2022

Hey @mcint , you're probably confused 😄 The repo for hackerweb.app is here instead: https://github.com/cheeaun/hackerweb

This repo (hackerweb-native-2) is for the native iOS mobile app.

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

No branches or pull requests

2 participants