ins elements: forces underline to be set below baseline to improve readability

[deniak]

fix #359

Moving the underline of ins elements below the baseline should improve readability

[daniel-montalvo]

Thanks much @deniak

I am approving as I understand this is an improvement with respec to what we currently have. Just pinging also @kfranqueiro for the visual check :-)

[kfranqueiro]

Yes, I can confirm this seems to improve things.

I would also note that the problem being addressed seemed to be worse in Chromium-based browsers than in Firefox-based browsers or Safari due to the dashed underline being thicker by default, so I wonder if setting text-decoration-thickness: 1px; would be useful in addition to this change?

[deniak]

Yes, I can confirm this seems to improve things.

I would also note that the problem being addressed seemed to be worse in Chromium-based browsers than in Firefox-based browsers or Safari due to the dashed underline being thicker by default, so I wonder if setting text-decoration-thickness: 1px; would be useful in addition to this change?

Yes, I does seem to improve readability on chromium-based browsers but in Firefox, it actually increases the default thickness. I pushed a version with the value from-font which seems to render better across the different browsers.