Skip to content

Web terminal is not matching the current site theme #3305

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

Closed
BrunoQuaresma opened this issue Jul 29, 2022 · 3 comments · Fixed by #3694
Closed

Web terminal is not matching the current site theme #3305

BrunoQuaresma opened this issue Jul 29, 2022 · 3 comments · Fixed by #3694
Labels
site Area: frontend dashboard
Milestone

Comments

@BrunoQuaresma
Copy link
Collaborator

Screen Shot 2022-07-29 at 16 00 30

I'm not sure how hard it is to make the theme match the current site colors, or if we could choose an existing one that is more "purple-ish" or "blue-ish".

@BrunoQuaresma BrunoQuaresma added the site Area: frontend dashboard label Jul 29, 2022
@mattlqx
Copy link

mattlqx commented Aug 12, 2022

Is it feasible to have this customizable? I'd care more about custom font and font size though.

@BrunoQuaresma
Copy link
Collaborator Author

I think the font size is, but not sure about the custom font since the user would have to upload that. Maybe we could provide some built-in fonts? Also, where would you add the terminal settings?

@mattlqx
Copy link

mattlqx commented Aug 12, 2022

I humbly suggest https://github.com/tonsky/FiraCode 🙂 (it has some nice ligatures for arrows, equalities, etc.)

I think the presently spartan Account Settings page would be great to have some user preferences when it comes to things like defaults and settings like this. A subtab for Terminal could house these nicely.

@kylecarbs kylecarbs added this to the EE milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site Area: frontend dashboard
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants