Skip to content

Commit ca66074

Browse files
authored
Merge pull request #4106 from grandixximo/docs/copy-linked-images
docs: copy click-to-enlarge images, not just displayed ones
2 parents d9d6c52 + 48bbfa0 commit ca66074

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

docs/src/Submakefile

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1001,13 +1001,17 @@ $(foreach L,$(LANGUAGES),$(eval $(call HTML_COPY_RULE,$(L))))
10011001
# $(DOC_OUT_HTML)/<lang>/<topic>/X.html. The source images are in
10021002
# $(DOC_SRCDIR)/<topic>/ regardless of language (translations are
10031003
# image-symlinked to English originals via the sed below).
1004+
#
1005+
# src="..." covers displayed images; href="...png" covers click-to-enlarge
1006+
# targets from image:thumb[link=...] (which would otherwise 404). The
1007+
# extension filter keeps page/anchor hrefs out.
10041008
.html-images-stamp: $(DOC_TARGETS_HTML)
10051009
set -e; for HTML_FILE in $^; do \
10061010
HTML_REL=$$(echo $$HTML_FILE | sed 's%^$(DOC_OUT_HTML)/%%'); \
10071011
LANG=$$(echo $$HTML_REL | cut -d/ -f1); \
10081012
REST=$$(echo $$HTML_REL | cut -d/ -f2-); \
10091013
HTML_DIR=$$(dirname $$REST | cut -d/ -f1); \
1010-
for IMAGE_FILE in $$(grep -oE 'src="[^"]+"' $$HTML_FILE | sed 's/src="//;s/"$$//' | grep -vE '^https?:|^data:|^/'); do \
1014+
for IMAGE_FILE in $$(grep -oE '(src|href)="[^"]+"' $$HTML_FILE | sed -E 's/^(src|href)="//;s/"$$//' | grep -vE '^https?:|^data:|^/' | grep -iE '\.(png|jpe?g|gif|svg|webp)$$'); do \
10111015
IMAGE_DIR=$$(dirname $$IMAGE_FILE); \
10121016
IMAGE_PATH=$(DOC_SRCDIR)/$$HTML_DIR/$$IMAGE_FILE; \
10131017
mkdir -p $(DOC_OUT_HTML)/$$LANG/$$HTML_DIR/$$IMAGE_DIR; \

0 commit comments

Comments
 (0)