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 trac header and footer for updated navigation link colors #89

Conversation

adamwoodnz
Copy link

@adamwoodnz adamwoodnz commented Jul 22, 2022

See WordPress/wporg-mu-plugins#218

The link colors for the global navigation on wordpress.org sites were updated in WordPress/wporg-mu-plugins#214

This PR updates the trac headers to take these changes.

I ran https://github.com/WordPress/wordpress.org/blob/trunk/trac.wordpress.org/templates/update-headers.php to make the updates as suggested.

@adamwoodnz adamwoodnz requested review from iandunn and ryelle July 22, 2022 04:03
@dd32
Copy link
Member

dd32 commented Jul 22, 2022

@adamwoodnz For sync things like this, you can just run the script and commit the diffs without needing to get a review, as long as you're not applying any customisations to it :)

@adamwoodnz
Copy link
Author

@adamwoodnz For sync things like this, you can just run the script and commit the diffs without needing to get a review, as long as you're not applying any customisations to it :)

Thanks, I struggled to find the update-headers.php script in my sandbox but I'll take another look

@dd32
Copy link
Member

dd32 commented Jul 22, 2022

I struggled to find the update-headers.php script in my sandbox

Ah, it doesn't exist, as not all files are checked out to sandboxes - you have to do a svn co https://meta.svn.wordpress.org/sites/trunk/trac.wordpress.org/templates/ and then run/commit it.

Because of that, I've just added bin/sync/trac-templates.sh just so we have all scripts in one place :)

@adamwoodnz adamwoodnz closed this Jul 22, 2022
@adamwoodnz adamwoodnz deleted the 218-fix-meta-trac-header-link-colors branch July 22, 2022 04:22
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