Skip to content
New issue

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

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Put English version first in list #42

Merged
merged 4 commits into from
Sep 16, 2016
Merged

Put English version first in list #42

merged 4 commits into from
Sep 16, 2016

Conversation

brunobord
Copy link
Owner

On the homepage, the translations are sorted alphabetically. This means that "brazilian-portuguese" will appear before the english source text.
Re-sort the language list before rendering using a trick to put english at the first place.

@brunobord brunobord added the bug label Sep 16, 2016
@brunobord brunobord added this to the Fall 2016 milestone Sep 16, 2016
@brunobord brunobord self-assigned this Sep 16, 2016
brunobord added a commit that referenced this pull request Sep 16, 2016
Incidentally, moved the Builder() class to the correct module
brunobord added a commit that referenced this pull request Sep 16, 2016
* ignoring .github directory
* pep8 on test_slugify.py
* drop pointless file (test that wasn't loaded)
Incidentally, moved the Builder() class to the correct module
@brunobord brunobord merged commit 10205f0 into master Sep 16, 2016
@brunobord brunobord deleted the 42-english-first branch September 17, 2016 19:07
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant