-
Notifications
You must be signed in to change notification settings - Fork 27
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
Retire OGM #163
Retire OGM #163
Conversation
%a.item{:href => "#{site.hibernate_org.base_url}ogm/"} OGM | ||
%a.item{:href => "#{site.hibernate_org.base_url}tools/"} Tools | ||
- if real_page.project == 'ogm' | ||
%a.item{:href => "#{site.hibernate_org.base_url}ogm/"} OGM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't understand this change. Why can't you just remove it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤷 That's what Guillaume did on hibernate.org, so I did the same here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Basically, we display the menu if people are in the OGM section already.
We do the same for the website.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I kinda get it why we would do that in hibernate.org.
But why do we need it for in.relation.to? If you click on any button of the header it will go to hibernate.org
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By the way, I miss this in hibernate.org, but OGM
should probably be after Others
. This way we can think of it as a sub-voice of Others
: Others > OGM
No description provided.