Skip to content

made update sidebar red with white color when it's critical update for slate theme#4706

Merged
TomJaeger merged 2 commits into7.devfrom
feature/7.x/change-color-for-critical-update
Mar 3, 2025
Merged

made update sidebar red with white color when it's critical update for slate theme#4706
TomJaeger merged 2 commits into7.devfrom
feature/7.x/change-color-for-critical-update

Conversation

@Yulyaswan
Copy link
Copy Markdown
Collaborator

Slate theme
Screenshot 2025-02-10 at 09 38 47

Snow theme (light)
Screenshot 2025-02-10 at 09 59 21

@TomJaeger TomJaeger added this to the 7.5.8 milestone Feb 13, 2025
@robinsowell
Copy link
Copy Markdown
Contributor

I hacked all over the place trying to get the cp reading that as a critical update. I can make it return red in the ajax little modal thing:
Screenshot 2025-02-20 at 3 50 17 PM

But never in the sidebar:

Screenshot 2025-02-20 at 3 50 25 PM

That was the case for all the themes. I'm either not triggering it to be 'vital' or they're all just not showing an indicator.

Controller/settings/general.php is the main place I changed it:

Screenshot 2025-02-20 at 3 52 57 PM

But I played around a lot. Need to figure out how to set it and know it's set as a 'critical update' to test.

@robinsowell
Copy link
Copy Markdown
Contributor

Well mine was weird, but I paired with @bakin1999 and screen share shows all is well! Approved.

@bakin1999 bakin1999 self-requested a review February 27, 2025 20:55
Copy link
Copy Markdown
Collaborator

@bakin1999 bakin1999 left a comment

Choose a reason for hiding this comment

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

worked correctly for me

@TomJaeger TomJaeger merged commit 0b41aa1 into 7.dev Mar 3, 2025
@TomJaeger TomJaeger deleted the feature/7.x/change-color-for-critical-update branch March 3, 2025 20:09
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.

4 participants