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
Baguette in bootstrap #2929
Baguette in bootstrap #2929
Conversation
<!-- fancy dates --> | ||
<script> | ||
moment.locale("${momentjs_locales[lang]}"); | ||
fancydates(${date_fanciness}, ${js_date_format}); | ||
</script> | ||
<!-- end fancy dates --> | ||
<%block name="extra_js"></%block> | ||
<script>baguetteBox.run('a.reference', { |
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.
Why did we change the classes baguetteBox works with?
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 copied from base theme. Probably both should be what was in bootstrap instead.
I'll take a look at it tomorrow.
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.
Ping?
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.
This seems to work in the demo cases. Since baguette is not JQuery based, I don't see a simple way to use that complex selector :-(
Sorry am sorta swamped. I'll take a look this week-ish.
…On Mon, Dec 11, 2017 at 2:23 PM Chris Warrick ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In nikola/data/themes/bootstrap3/templates/base.tmpl
<#2929 (comment)>:
> <!-- fancy dates -->
<script>
moment.locale("${momentjs_locales[lang]}");
fancydates(${date_fanciness}, ${js_date_format});
</script>
<!-- end fancy dates -->
<%block name="extra_js"></%block>
+ <script>baguetteBox.run('a.reference', {
Ping?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#2929 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAAGK4m5v0P3By2I5qoLs7Iy21KZAnrJks5s_WUigaJpZM4Qz-_o>
.
|
Please backport the |
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.
Okay, so .islink
is kinda different, but eh, it’s good enough.
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Thanks for merging and reviewing @Kwpolska |
This change is backwards-incompatible since it's bound to break people's themes.