When the figure helper is used in a list, which can be either ordered or
unordered, and no caption is specified, a line with text "</figure>"
will be shown below the figure on the rendered page.
This is because, if the '{% if include.caption %}' evaluates to false,
the lines between that 'if' statement and '{% endif %}' will be emptied,
not removed, so the block will be filled by empty lines.
HTML ignores redundant empty lines, but Markdown takes them seriously.
In addition, Markdown expects proper indentation of lines inside lists,
and the closing '</figure>' tag is not indented. When combined, the
empty space and absence of indentation cause Markdown to process the
'</figure>' tag as a separate paragraph instead of an HTML tag, thus the
text for the tag is directly rendered on the page.
The fix for this issue is very simple: remove the empty space when
'include.caption' is false. As described in
<https://shopify.github.io/liquid/basics/whitespace/>, this can be done
by adding hyphens to the 'if' and 'endif' tags.
* Rename include
* Add grid view test pages
* Rename `.post__meta-sep` and use CSS to add line break
* Improve collection grid archive
* Improve page grid archive
* Enable `grid`
* Don't show date icon if there is no `date` value
* Add blank line at EOF
* Add space
* Wrap date and reading time in named `span` elements
* add date to read-time.html
* add option for show_date, dynamic icon style
* change read-time to post__meta
* cleanup post__metal.html
* cleanup post__meta include variables
* put date before read time
* remove space in include variable
* allow customisation of post__meta separator
* add some documentation
* oops fix typo derp
* add post date image
* change page meta separator customisation to CSS
* Enhance support for bilibili videos in responsive video helper, and add corresponding doc
* Apply @iBug's review in mmistakes/minimal-mistakes#2522
* Fix danmaku in page hero video
* Update video to use case..when for iframe src
* Update CHANGELOG and history
Co-authored-by: Anran <51769728+ALeafWolf@users.noreply.github.com>
Co-authored-by: Michael Rose <mmistakes@users.noreply.github.com>
* Search on menu toggle
With that change search is performed when requested. Before it was done
for each pageview, regardless of the fact if search was used by user or
not.
* Localize algolia's empty template
* Use relative_url and absolute_url where possible
Drops the `contains "://"` check, adopt Jekyll 3.7
Ref: https://github.com/mmistakes/minimal-mistakes/pull/2385#issuecomment-579882236
* One more unneeded {% assign %}
* Remove one more assign as noted by mmistakes
* Consolidate 4 more captures
* Consolidate an extra assign on "active" class
* enable markdown in author bio
added the markdownify liquid filter
* Add markdown to bio in _config.yml
* add markdown to bio in /test/ _config.yml
* Change <p> to <div>
* Fixes aria issues on search form
* Swapping aria-label for <label> tag in search
* Removing background gray caused by adding <form> tag to search
* Removing redundant space
* Making form not submit if key is enter
Close#2180
* Add site subtitle
* Tabs vs spaces - the ancient conflict
* updates cfr PR #2205
* Perhaps also add the closing anchor
* Make sure we check for the right variables...
* Brown paper bag - subtitle, not description
* Update search_form.html
Updates Issue #2122 by adding the correct input type and aria label.
* update search_form.html
should be area-placeholder. My mistake. Updates Issue #2122
* fix aria-placeholder
Updates issue #2123 with a typo found by @mmistakes for liquid syntax.
Close#2122