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

CSS content update: learn to support older browsers #31473

Merged
merged 11 commits into from
Jan 12, 2024
Merged

Conversation

estelle
Copy link
Member

@estelle estelle commented Jan 3, 2024

remove grid / float fallback example (outdated)
add lingo about evergreen
add browser error handling
change PNG to include a BCD live example instead

In combo with #31387 and openwebdocs/project#188

@estelle estelle requested a review from a team as a code owner January 3, 2024 00:16
@estelle estelle requested review from dipikabh and removed request for a team January 3, 2024 00:16
@estelle estelle marked this pull request as draft January 3, 2024 00:16
@github-actions github-actions bot added the Content:Learn:CSS Learning area CSS docs label Jan 3, 2024
@estelle estelle changed the title CSS content update: learn to support older browsers Draft: CSS content update: learn to support older browsers Jan 3, 2024
@estelle estelle marked this pull request as ready for review January 3, 2024 00:17
Copy link
Contributor

github-actions bot commented Jan 3, 2024

Preview URLs

External URLs (1)

URL: /en-US/docs/Learn/CSS/CSS_layout/Supporting_Older_Browsers
Title: Supporting older browsers

(comment last updated: 2024-01-12 22:46:01)

@dipikabh dipikabh self-assigned this Jan 5, 2024
@dipikabh
Copy link
Contributor

dipikabh commented Jan 5, 2024

Hi @estelle, let me know if this is ready to review. The title says "Draft"

@estelle estelle changed the title Draft: CSS content update: learn to support older browsers CSS content update: learn to support older browsers Jan 9, 2024
@estelle
Copy link
Member Author

estelle commented Jan 9, 2024

@dipikabh This is ready. It could likely be deleted altogether, but that would require changes to every page in the CSS layout section, so I edited it instead.

Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

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

Hi @estelle, I finished a readthrough of the changed text. I've added my suggestions and some questions.

estelle and others added 2 commits January 10, 2024 18:00
Co-authored-by: Dipika Bhattacharya <[email protected]>
Co-authored-by: Dipika Bhattacharya <[email protected]>
estelle and others added 2 commits January 10, 2024 18:17
…ex.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@estelle estelle requested a review from dipikabh January 12, 2024 00:03
Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

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

Almost there, just a few more edits and flaws to be fixed


{{ EmbedLiveSample('Falling_back_from_grid_to_float', '100%', '200') }}
These techniques were demonstrated in the [legacy techniques layout tutorial](/en-US/docs/Learn/CSS/CSS_layout/Legacy_Layout_Methods). Back when grid wasn't fully supported, the browser ignored the grid styles. This tutorial is now a good demonstration of how newer features, in this case grid layout, are designed and defined to take precedence over older features, floats in this case, when their effects would have otherwise conflicted.
Copy link
Contributor

Choose a reason for hiding this comment

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

I guess we removed this para?

Co-authored-by: Dipika Bhattacharya <[email protected]>
@estelle
Copy link
Member Author

estelle commented Jan 12, 2024

thanks @dipikabh

Also, yes, I removed the paragraph linking to legacy layouts as the explanation that was the reason for the link is no longer on that page.

@estelle estelle requested a review from dipikabh January 12, 2024 20:10
Copy link
Contributor

@dipikabh dipikabh left a comment

Choose a reason for hiding this comment

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

Thanks for your work on updating this page, @estelle 🎉

@dipikabh dipikabh merged commit e308781 into mdn:main Jan 12, 2024
7 checks passed
dipikabh added a commit to dipikabh/content that referenced this pull request Jan 17, 2024
* supporting older browsers page

* remove more legacy cruft

* typo

* changes the entire see also

* Apply suggestions from code review

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Apply suggestions from code review

Co-authored-by: Dipika Bhattacharya <[email protected]>

* Update files/en-us/learn/css/css_layout/supporting_older_browsers/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* rewrote selector section

* Apply suggestions from code review

Co-authored-by: Dipika Bhattacharya <[email protected]>

* fix typo

---------

Co-authored-by: Dipika Bhattacharya <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@estelle estelle deleted the guides branch March 26, 2024 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:Learn:CSS Learning area CSS docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants