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
Import Tabzilla into Bedrock #507
Conversation
import l10n_utils | ||
|
||
|
||
def tabzilla_js(request): |
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.
You should probably add a cache decorator to get zeus to cache this for some amount of time to avoid extra load.
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.
All our HTTP requests are cached 10 minutes by default.
r+ for what's here. The LESS variable is a nice-to-have and we could live with an added step to the docs if you're strapped for time. Let me know when you add the other stuff so I can finish the review. :D |
r+ |
Pull request updated with documentation imported. |
|
||
*tabzilla* is the universal tab displayed on Mozilla websites. | ||
|
||
Adding the universal tab requires to a site requires: |
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.
Extra "requires".
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.
Fixed.
r+ looks good to me |
- Make it localisable - Add redirects for backwards compatibility - Adds a js_escape filter bug 744271
I'm submitting this to get the code approved. Do not merge yet ;)
Things left to do:
tabzilla/media/js/tabzilla.js
,tabzilla/media/css/tabzilla.css
andtabzilla/tabzilla.js
.