søn. 24. jan. 2021 kl. 19:16 skrev Holger Wansing <hwans...@mailbox.org>: > > Hi, > > Holger Wansing <hwans...@mailbox.org> wrote: > > Thomas Lange <la...@cs.uni-koeln.de> wrote: > > > >>>>> On Tue, 19 Jan 2021 21:53:28 +0100, Holger Wansing > > > >>>>> <hwans...@mailbox.org> said: > > > > > > > Sounds good. > > > > I would vote for the 'important' class (the exclamation mark > > > attracts more > > > > attention IMO). > > > > > > > Patch updated. > > > Thanks. > > > "... about loading firmware during an installation can be found in the > > > Installation Guide." > > > > > > Shouldn't we directly link to the section how to load the missing > > > firmware instead of just on the main page of the Installation guide > > > and then our end users need to search the correct section? > > > It's this URL > > > https://www.debian.org/releases/stable/amd64/ch06s04.en.html > > > > Basically, yes. > > However, I have modified it to use the URL > > https://www.debian.org/releases/stable/amd64/ch06s04 > > because that gives automatic use of CN, to get the installation-guide in > > the correct language. > > (Only on the devel/debian-installer page I link statically to the english > > version, because CN does not work there, due to the directory structure on > > dillon). > > I have added hints for translators, to make them aware of the fact, that the > paragraph is existing several times in webwml.
Looks good to me. One suggestion: Replace https://cdimage.debian.org/cdimage/unofficial/non-free/firmware/buster/current/ with https://cdimage.debian.org/cdimage/unofficial/non-free/firmware/stable/current/ for easier maintenance? Regards, Hans