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

Make the announcements section easy to be configured #327

Merged
merged 2 commits into from Nov 27, 2018

Conversation

dpordomingo
Copy link
Contributor

fix #325

The landing main announcement can be changed modifying hugo.config.yaml::announcement (here)

If no title or url is provided for the announcement, it wont be shown at the landing (leaving an empty white row in that place)

Here is a screenshot with the purpose made by @campoy at #325 (comment)

image

Signed-off-by: David Pordomingo <David.Pordomingo.F@gmail.com>
Copy link
Contributor

@campoy campoy left a comment

Choose a reason for hiding this comment

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

+1 but should one should review it

src/sass/partials/_layout.scss Outdated Show resolved Hide resolved
Signed-off-by: David Pordomingo <David.Pordomingo.F@gmail.com>
@dpordomingo
Copy link
Contributor Author

@campoy your suggestion was addressed.

But I didn't understood your suggestion about «one reviewing it»
Should we wait for any other review?

@campoy
Copy link
Contributor

campoy commented Nov 26, 2018

I'm not an expert on frontend tech, so it's probably worth having someone else reviewing the CSS etc.

@dpordomingo
Copy link
Contributor Author

The CSS and HTML are the.same than the original ones, so if that's your concern, I think we could merge.
In this case, what you should review is only the new way to modify the announcement.

@campoy campoy merged commit 79f1625 into src-d:master Nov 27, 2018
@dpordomingo dpordomingo deleted the news-section branch January 14, 2019 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change NEW announcement to point to the Engine video post
2 participants