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

Webview design cleanup #170

Merged
merged 9 commits into from
Mar 17, 2021
Merged

Webview design cleanup #170

merged 9 commits into from
Mar 17, 2021

Conversation

itsmesean
Copy link

🦢 🎵

Just a few tweaks to ui

@itsmesean itsmesean requested a review from mattseddon March 17, 2021 00:31
Copy link
Contributor

@rogermparent rogermparent left a comment

Choose a reason for hiding this comment

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

Thanks for this! We'll probably pick this PR apart as far as what fonts we use where, but it will still be useful as a starting point for discussion on where we want to go with changes in fonts and styling.

@@ -1,6 +1,7 @@
* {
box-sizing: border-box;
outline: none;
font-family: 'monospace', 'Segoe UI', Tahoma, Geneva, Verdana, sans-serif;
Copy link
Contributor

Choose a reason for hiding this comment

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

We'll probably want to use some of the VSCode-injected CSS variables like --vscode-font-family, --vscode-font-weight, and --vscode-editor-font-family so we use the user's configured fonts- doing so allows us to integrate our webviews with the user's VSCode and take advantage of the fonts provided to us as a free resource.

I'm thinking since the --vscode-font-family is generally sans serif and --vscode-editor-font-family monospace, we can use the former for UI elements and the latter for the table.
This concern relates to #175 as well.

@rogermparent rogermparent mentioned this pull request Mar 17, 2021
13 tasks
Copy link
Member

@mattseddon mattseddon left a comment

Choose a reason for hiding this comment

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

we will iterate on this

@mattseddon mattseddon merged commit 61b1dcd into master Mar 17, 2021
@mattseddon mattseddon deleted the webview-design-cleanup branch March 17, 2021 21:57
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.

3 participants