Skip to content

Conversation

@clesaec
Copy link
Contributor

@clesaec clesaec commented Sep 15, 2022

What is the purpose of the change

*For AVRO-3624 Try to complete site with Apache references to get greener note in https://whimsy.apache.org/site/.

Verifying this change

Manual check :
on folder "doc", run hugo via command hugo server --buildDrafts --buildFuture --bind 0.0.0.0 --navigateToChanged

This change is only on documentation.
I tried to have somethink that look like "ASF Links" on Apache arrow site, but i succeed only with HTML select/option code, so it's little weird.

Documentation

  • Does this pull request introduce a new feature? no

Copy link
Contributor

@RyanSkraba RyanSkraba left a comment

Choose a reason for hiding this comment

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

In principle, this LGTM -- the SELECT style is a bit bizarre, but I really have no objection to having someone design-oriented come in and clean it up for us. There's a lot of UX improvements that could be done on the site!

Bear with me while I try and figure out the right way to deploy changes between releases!

@martin-g
Copy link
Member

I will try to re-work the select to a menu (as on Arrow site) soon!

Move "ASF links" to the right and render it as a dropdown menu

Signed-off-by: Martin Tzvetanov Grigorov <[email protected]>
@martin-g
Copy link
Member

I will try to re-work the select to a menu (as on Arrow site) soon!

Done with b032e71

@martin-g martin-g merged commit eb2b19d into apache:master Sep 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants