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

Update and-or.css #2219

Merged
merged 4 commits into from
Aug 15, 2023
Merged

Update and-or.css #2219

merged 4 commits into from
Aug 15, 2023

Conversation

Christopher-O
Copy link
Contributor

Updated CSS to allow brdr-0 to exist on all resolutions when the design is stacked, and not just "always stacked".

Updated CSS to allow brdr-0 to exist on all resolutions when the design is stacked, and not just "always stacked".
@Christopher-O Christopher-O temporarily deployed to github-ci August 2, 2023 21:51 — with GitHub Actions Inactive
@Christopher-O Christopher-O marked this pull request as ready for review August 2, 2023 21:52
@duboisp
Copy link
Member

duboisp commented Aug 7, 2023

Pre-approved upon review and after confirming there is an example of this change.

This was referenced Aug 7, 2023
@duboisp
Copy link
Member

duboisp commented Aug 7, 2023

pre-approved upon testing

Relates to

@Christopher-O
Copy link
Contributor Author

@duboisp there is an example of the update on index.html under the updated header of no border.

Updated examples to show that brdr-0 can now be added to any side by side version and the border left will disappear when stacked.
duboisp
duboisp previously approved these changes Aug 15, 2023
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested the CSS change which are demoed in PR 2220 and work as expected. All CSS is demoed as expected.

Todo:

  • Merge all other PRs into a single one.

@duboisp duboisp temporarily deployed to github-ci August 15, 2023 03:35 — with GitHub Actions Inactive
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix the HTML markup in the edge-cases.html. The <s> element can only be use in phrasing context according to HTML 5 spec.

Other than that, the rest is good to be squash and merge

méli-mélo/2021-05-conjunction/edge-cases.html Outdated Show resolved Hide resolved
@duboisp duboisp temporarily deployed to github-ci August 15, 2023 19:38 — with GitHub Actions Inactive
Copy link
Member

@duboisp duboisp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and reviewed locally

@duboisp duboisp merged commit 8b7e660 into wet-boew:master Aug 15, 2023
1 check passed
SebastianBurke pushed a commit to SebastianBurke/GCWeb that referenced this pull request Mar 6, 2024
…ew#2219)

* Update and-or.css

Updated CSS to allow brdr-0 to exist on all resolutions when the design is stacked, and not just "always stacked".

* Update index.html

Updated examples to show that brdr-0 can now be added to any side by side version and the border left will disappear when stacked.

* Update edge-cases.html

* méli-mélo 2021-05-conjunction - Content adjustment according to HTML5 parsing


Patch - Méli-mélo 2021-05-conjunction

---------

Co-authored-by: Pierre Dubois <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants