Skip to content
This repository has been archived by the owner on Aug 13, 2023. It is now read-only.

Add news post for Playtest 20190825 #454

Merged
merged 1 commit into from
Aug 25, 2019
Merged

Add news post for Playtest 20190825 #454

merged 1 commit into from
Aug 25, 2019

Conversation

pchote
Copy link
Member

@pchote pchote commented Aug 24, 2019

Screenshot_2019-08-24 OpenRA

Screenshot 2019-08-24 at 21 10 48

@reaperrr
Copy link
Contributor

Didn't notice any issues apart from the TODOs.

@pchote
Copy link
Member Author

pchote commented Aug 25, 2019

Finished TODOs.

@pchote
Copy link
Member Author

pchote commented Aug 25, 2019

Fixed.

reaperrr
reaperrr approved these changes Aug 25, 2019
View reviewed changes
Copy link
Contributor

@reaperrr reaperrr left a comment

Choose a reason for hiding this comment

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

Gave it another read, looks good to me

@pchote pchote merged commit 63f5678 into OpenRA:master Aug 25, 2019
@anjew175
Copy link

Do you think we should use the 'about-highlight' css class instead of 'color: #d92c2c' to highlight the words in the newspost consistently with how the site is now? Right now every page except the front page is using a yellow highlight after #453

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@teinarss teinarss teinarss left review comments

@reaperrr reaperrr reaperrr approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@pchote @reaperrr @anjew175 @teinarss