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

Remove non-styling IDL CSS classes #1917

Open
saschanaz opened this issue Nov 28, 2018 · 2 comments
Open

Remove non-styling IDL CSS classes #1917

saschanaz opened this issue Nov 28, 2018 · 2 comments
Labels

Comments

@saschanaz
Copy link
Collaborator

#1892 removed many CSS classes, but we still have some that dynamically added for test purpose, e.g. idlInterface. Currently they are to make sure we get the correct IDL text, but that purpose can be fulfilled more easily by comparing the whole processed textContent of <pre> with the original untouched one.

What do you think? @marcoscaceres

@marcoscaceres
Copy link
Contributor

marcoscaceres commented Nov 28, 2018

Removing them is reasonable so long as we can continue styling things consistently.

@ridhishjain-zepto
Copy link

@saschanaz, I got what you've done in #1892, however, the directory structure is changed a lot since then.
can u tell me where to look for the processing part of textContent you've mentioned?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants