Skip to content

Upgrade materialize#3594

Merged
mauromsl merged 13 commits into
masterfrom
3425-upgrade-materialize
Sep 15, 2023
Merged

Upgrade materialize#3594
mauromsl merged 13 commits into
masterfrom
3425-upgrade-materialize

Conversation

@joemull

@joemull joemull commented Jun 2, 2023

Copy link
Copy Markdown
Member

Closes #3425

This morning we discussed potentially keeping this out of 1.5.1 in case there are problems with custom CSS that different journals have implemented using selectors from Materialize 0.100.2.

Based on the v1 upgrade guide, there are only a few classes and attributes that are likely to need attention. The following were all changed one-to-one:

Classes

dropdown-button was changed to dropdown-trigger
button-collapse was changed to sidenav-trigger
side-nav was changed to sidenav

Data attributes

data-activates was changed to data-target

When we release this upgrade, we should encourage press managers to check their custom stylesheets for any affected selectors and to change them to the new ones.

@joemull joemull requested review from ajrbyers and mauromsl June 2, 2023 14:28
@joemull

joemull commented Jun 2, 2023

Copy link
Copy Markdown
Member Author

Note to self:

  • Add a migration to change the affected selectors in custom stylesheets

@mauromsl mauromsl merged commit a70785a into master Sep 15, 2023
@mauromsl mauromsl deleted the 3425-upgrade-materialize branch September 15, 2023 14:37
ajrbyers added a commit that referenced this pull request Mar 13, 2024
… rather than using stages, will also reduce the number of queries.
ajrbyers added a commit that referenced this pull request Mar 19, 2024
… rather than using stages, will also reduce the number of queries.
ajrbyers added a commit that referenced this pull request May 10, 2024
… rather than using stages, will also reduce the number of queries.
mauromsl pushed a commit that referenced this pull request May 14, 2024
… rather than using stages, will also reduce the number of queries.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Migrate Materialize Django helper

3 participants