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

Maybe remove mapping-tables.css #102

Open
spectranaut opened this issue Sep 4, 2023 · 2 comments
Open

Maybe remove mapping-tables.css #102

spectranaut opened this issue Sep 4, 2023 · 2 comments
Assignees

Comments

@spectranaut
Copy link
Contributor

Hey @pkra in reply to your comment... Somehow I thought that it was still necessary to keep mapping-tables.css, and I just checked, at this point in only makes the tables the same width. Otherwise they are the width of the text, so each table is a different size which imo doesn't look that great. But maybe we can move that css to common.css?

I wish I had thought to actually look at that file when doing the work to remove the javascript file, lol.

@pkra
Copy link
Member

pkra commented Sep 5, 2023

Thanks for clarifying. I had erroneously thought that the css was for the large table.

But maybe we can move that css to common.css?

👍 to that.

@spectranaut spectranaut self-assigned this Oct 16, 2023
@spectranaut
Copy link
Contributor Author

also maybe fix the widths in the notification tables

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