Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Re-structure and simplify menu layout #207

Merged
merged 1 commit into from Jun 19, 2013

Conversation

Projects
None yet
2 participants
Contributor

saivann commented Jun 14, 2013

This pull request merges all menus together in an effort to simplify the layout and make it less cluttered. It can be previewed live here : http://bitcoinmenu.us.to/en

The following pages are renamed (redirected) :

bitcoin-for-press - becomes press and is moved in resources
bitcoin-for-enthusiasts - becomes innovation and gets more visibility.
resources - becomes links

The new Innovation page could qualify for a complete rewrite focused on explaining the real innovative nature of Bitcoin. Everything that can start with "it is the first time in history.." would fit there. With a few short insights of the potential and impact it could have.

The About page should be replaced (and recycled) by a FAQ in the future.

Compatibility

This layout remains compatible with IE6, mobiles and all browsers that were supported.

Progressive deployment in translations

This pull request also implement a new _redirects.yml file that allows to interfere with template.rb plugin in order to generate page redirections for selected pages and languages. In such a way that any change in website structure can be deployed progressively in translations without breaking layouts, page urls and sitemap.

Contributor

mikehearn commented Jun 17, 2013

Compatibility with IE6 is brave :) I don't think it's worth spending time on that though unless it's very easy because hardly anyone uses it any more.

The new menu looks great, I was thinking we'd need to move to a dropdown system eventually. The only issue is that "Presentation" is kind of a weird label for that collection of items. Maybe "Learn" instead?

Contributor

saivann commented Jun 17, 2013

Thanks :)

IE6 compatibility was indeed worthless until we had Chinese translations ongoing. 24% of Chinese users still use IE6. http://www.ie6countdown.com/ The whole website has been made compatible with IE6 already.

I wasn't sure about "Presentation". "Learn" sounds like this content is a complete education resource. But it is more a very short introduction to the useful properties and limitations of Bitcoin for each new user category. It could work better than Presentation, but I have a preference to what sounds like "Introduction to Bitcoin", any more idea?

Ideally, I think that "What is Bitcoin" is the best representation for this dropdown menu. But I wished we could find a "one word description" to keep the layout clean.

Contributor

saivann commented Jun 17, 2013

Would it make sense to use "Introduction"? I suspect that this would also be easier to translate.

capture du 2013-06-17 15 33 30

Contributor

mikehearn commented Jun 17, 2013

Yeah, that's a good suggestion.

On Mon, Jun 17, 2013 at 9:48 PM, saivann notifications@github.com wrote:

Would it make sense to use "Introduction"? I suspect that this would also
be easier to translate.

[image: capture du 2013-06-17 15 33 30]https://f.cloud.github.com/assets/3578089/664501/a3eae738-d786-11e2-95f7-58d0bf689c39.png


Reply to this email directly or view it on GitHubhttps://github.com/bitcoin/bitcoin.org/pull/207#issuecomment-19569765
.

Re-structure and simplify menu layout (fixes #198)
Add translated page redirections using _redirects.yml
Redirect for bitcoin-for-press to press
Redirect bitcoin-for-enthusiasts to innovation
Ignore page redirections in sitemap.rb plugin
Move template plugin to template.rb instead of translate.rb
Add a small documentation in the template plugin

saivann added a commit that referenced this pull request Jun 19, 2013

Merge pull request #207 from bitcoin/simplemenu
Re-structure and simplify menu layout

@saivann saivann merged commit 45c360b into master Jun 19, 2013

@saivann saivann deleted the simplemenu branch Jun 19, 2013

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