-
Notifications
You must be signed in to change notification settings - Fork 740
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
Style visual test app views #1181
Conversation
Rationale for using
|
@ashmaroli Can you provide a bit more context to explain the purpose of this PR? Is it just to enhance readability? I'm not sure I see a large difference between your screenshots and how it looks now. |
@pyrmont The purpose of this PR is to improve the look of the visual test views. There isn't a large difference between the views on I shall list everything that's different with this PR:
|
Line numbers for the raw sample sounds pretty sweet. |
@ashmaroli I'd like to run this locally for a test a bit later when I'm at home. Should be able to merge when that's done. |
Of course. There's no hurry. |
@ashmaroli Made some suggested fixes in ashmaroli#2 :) |
@ashmaroli Did you have anything else you wanted to add? I think this can be merged in. |
Previews
Index page
Lexer Page
/cc @jneen