Torna carregamento de imagens "lazy" por padrão (conversor customizado asciidoctor) #28
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Isso é feito sobrescrevendo os conversores de imagens do asciidoctor e adicionando o atributo "loading" que, quando não informado, possui o valor "lazy" por padrão.
arturo32
commented
Apr 22, 2023
Comment on lines
+9
to
+14
| # https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement/loading | ||
| loading_attr_value = (node.attr 'loading') || (node.document.attr 'image-loading') | ||
| if loading_attr_value && loading_attr_value != 'eager' && loading_attr_value != 'lazy' | ||
| logger.warn 'valid values for attribute "loading" are "eager" and "lazy". found: ' + loading_attr_value | ||
| end | ||
| loading_attr = loading_attr_value ? %( loading="#{loading_attr_value}") : '' |
There was a problem hiding this comment.
Pedaço de código que é diferente do código fonte da função convert_image do asciidoctor.
arturo32
commented
Apr 22, 2023
Comment on lines
+71
to
+76
| # https://developer.mozilla.org/en-US/docs/Web/API/HTMLImageElement/loading | ||
| loading_attr_value = (node.attr 'loading') || (node.document.attr 'image-loading') | ||
| if loading_attr_value && loading_attr_value != 'eager' && loading_attr_value != 'lazy' | ||
| logger.warn 'valid values for attribute "loading" are "eager" and "lazy". found: ' + loading_attr_value | ||
| end | ||
| attrs = %(#{attrs} loading="#{loading_attr_value}") if loading_attr_value |
There was a problem hiding this comment.
Pedaço de código que é diferente do código fonte da função convert_inline_image do asciidoctor.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Relacionado a issue #1:
Sobrescreve os conversores
convert_imageeconvert_inline_imagedo asciidoctor com base nesse PR do asciidoctor e nas recomendações de um dos desenvolvedores do asciidoctor nessa discussão .O PR citado adiciona praticamente 5 linhas nos conversores de imagens do asciidoctor, adicionando o atributo
loadingaos macrosimageeinline-imagedoasciidoctore o atributo globalimage-loading. Este último é o usado no arquivolivro.adoc.Copiei o código do PR e coloquei dentro de um conversor customizado do asciidoctor em ruby.
O conversor customizado é adicionado no comando de geração do arquivo HTML em
build.sh.Para que o lazy loading funcione é necessário que as imagens tenham suas origens definidas por links/caminhos. Logo o atributo global
:data:uri:teve que ser removido do arquivolivro.adoc.Vale notar que eu não tive experiência com a linguagem ruby anteriormente. Aprendi o básico da linguagem no rápido tutorial oficial da linguagem e em outras fontes.