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.

Sign up for GitHub

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

Jump to bottom

Combine CSS into a separate file #32

Closed
easytarget opened this issue Sep 10, 2020 · 1 comment · Fixed by #36
Closed

Combine CSS into a separate file #32

easytarget opened this issue Sep 10, 2020 · 1 comment · Fixed by #36
Assignees
@easytarget
Labels
enhancement New feature or request
Milestone
V3

Comments

@easytarget
Copy link
Owner

Currently each html doc (now three) has a huge CSS payload, it would make sense to serve this as a seperate CSS file to make the HTML and JS files smaller and easier to work with.
Bonus: makes theming easier since changes apply to all pages automagically.
@easytarget easytarget added the enhancement New feature or request label Sep 10, 2020
@easytarget easytarget added this to the V3 milestone Sep 10, 2020
@easytarget easytarget self-assigned this Sep 10, 2020
@easytarget
Copy link
Owner Author

There are some minor fixes from the miniviewer I want to bring across at the same time

@easytarget easytarget linked a pull request Sep 11, 2020 that will close this issue
move css into include file, and reduce whitespace in hosted site. #36
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@easytarget easytarget

Labels
enhancement New feature or request
Projects
None yet
Milestone
V3
Development

Successfully merging a pull request may close this issue.

move css into include file, and reduce whitespace in hosted site. easytarget/esp32-cam-webserver
1 participant
@easytarget