frontend: improvements to the default "how to install" layout #1003
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I've seen people complain that `nix-env` is a weird "default" option to install packages with, particularly because of its weird ergonomics: - Install packages with `nix-env -iA nixos.hello` - Uninstall without the `nixos` prefix I think the community consensus has been to avoid the `nix-env` command altogether, but at first glance, to a newcomer, since the `nix-env` tab is currently the leftmost option (although not the one selected by default), `nix-env` might look like the way to go... To make the install instruction page slightly more ergonomic (and hopefully nudge people to the correct option when they start out), I've ordered them from left-to-right (read: reversed), and greyed out the `nix-env` option, making it the least prominent from now on visually. Now this might entail the fact that some screenreaders and other structured consumers of the markup will see the options the wrong way around, but I guess that's just a flaw of the design, where `pull-right` is used to right-align the tabs, incidentally reversing the order.... Disclaimer: I don't have any experience with Elm so forgive me if I struggle to follow-up further review comments or requests.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See the message at f8d9c2c for motivation behind this change.