Skip to content
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

Translation of the 2020 mobile web chapter to Dutch #2064

Merged
merged 8 commits into from
Mar 12, 2021
Merged

Translation of the 2020 mobile web chapter to Dutch #2064

merged 8 commits into from
Mar 12, 2021

Conversation

strangernr7
Copy link
Contributor

#1750
Kept mSpeed as is but I can change it if needed

@strangernr7 strangernr7 added the translation world wide web label Mar 11, 2021
@strangernr7 strangernr7 added this to the 2020 Content Translation milestone Mar 11, 2021
Copy link
Member

@tunetheweb tunetheweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spotted a few issues

src/content/nl/2020/mobile-web.md Outdated Show resolved Hide resolved
src/content/nl/2020/mobile-web.md Outdated Show resolved Hide resolved
src/content/nl/2020/mobile-web.md Outdated Show resolved Hide resolved
src/content/nl/2020/mobile-web.md Outdated Show resolved Hide resolved
src/content/nl/2020/mobile-web.md Outdated Show resolved Hide resolved
@tunetheweb tunetheweb merged commit 3a7db06 into HTTPArchive:main Mar 12, 2021
@strangernr7 strangernr7 deleted the mob_web-2020-nl branch March 12, 2021 10:15
@strangernr7
Copy link
Contributor Author

And I can't see any reason for the extra alt so can we remove that (in English too if possible):

So in future cases, if there's an alt and a caption we just remove the alt?

@tunetheweb
Copy link
Member

tunetheweb commented Mar 12, 2021

The caption is used as the alt, when it's not explicitly provided.

The caption is what is displayed on screen (and also what's used by screen readers since we expose that with aria-labelledby which overrides the alt).

The alt is used when image fails to download and also for SEO purposes — though with a decent filename it's probably a bit redundant to be honest, but we didn't use decent filenames for the 2019 edition (we got better this year).

The reason we have both is to allow them to be different if they need to be. Why would you want a different alt? For a few reasons:

  1. If the caption contains HTML (e.g. code formatting, or <span lang="en"></span> tags). There's some dispute whether an alt can have HTML or not, but I agree with this answer and think not, but caption can have it.
  2. Kind of an extension to that, if the caption contains a source reference, then you could remove the link from the alt (example). The mobile web uses them but as plain text (example). Now you've made me think about this some more, I think it should the best answer is a combination of both - have the plain text source as the alt, but then the hyperlinked version as the caption.
  3. Something else I've not thought of yet :-)

So yes if the alt and the caption are exactly the same (as there were in a couple of places here), then there is no benefit to having both as the template will automatically copy the caption into the alt and so I think we should remove to avoid duplication and having to translate the same thing twice.

Make sense?

All this is getting very picky of me btw! But good to have standards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation world wide web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants