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

editorial: revise custom stylesheets #2410

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

editorial: revise custom stylesheets #2410

wants to merge 7 commits into from

Conversation

pkra
Copy link
Member

@pkra pkra commented Jan 9, 2025

Resolves #2226


Preview | Diff

Removes unused and commented-out styles as per
#2226 (comment)
@pkra pkra added editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo Agenda-Editors labels Jan 9, 2025
Copy link

netlify bot commented Jan 9, 2025

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit c230c2c
🔍 Latest deploy log https://app.netlify.com/sites/wai-aria/deploys/677fbef3eb5c0c00084aa39a
😎 Deploy Preview https://deploy-preview-2410--wai-aria.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pkra
Copy link
Member Author

pkra commented Jan 9, 2025

notes to self:

Removes unused and commented-out styles as per
#2226 (comment)
- aria spec
  - rewrite content model figure to actual figure, using respec features
- common.css: remove .image and . img-caption
@pkra pkra self-assigned this Jan 9, 2025
@pkra
Copy link
Member Author

pkra commented Jan 9, 2025

It turns out, core-aam only sets styling that are already set in common.css.

pkra added 2 commits January 9, 2025 13:16
- remove core-aam.css
  - all styling is done identically in common.css
- core-aam/index.html
  - removes link to core-aam.css
Stylesheet was removed as all styling is done identically in common.css
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda-Editors editorial a change to an example, note, spelling, grammar, or is related to publishing or the repo spec:aria spec:core-aam spec:html-aam spec:pdf-aam
Projects
None yet
Development

Successfully merging this pull request may close these issues.

remove spec specific css or inline it
1 participant