Gerrit commit message body is always wrapped with MS Edge
Open, Needs TriagePublic
Actions

Assigned To

None

Authored By

	Xqt
	Jul 2 2020, 11:58 AM

Description

Gerrit commit message body is always wrapped with MS Edge 44.17763.1.0 and does not respect any line feeds.

Status	Assigned	Task
Resolved	QChris	T186406 Polygerrit Outgoing reviews, Incoming reviews and Recently closed headings should be more different from commits
Resolved	None	T220801 PolyGerrit 2.15 UI does not support custom dashboards (supported from in 2.16+)
Resolved	QChris	T206453 Big empty space under patch description
		Restricted Task
Resolved	None	T210515 context links in diff don't use pointer cursor
Resolved	Paladox	T223278 "Server error: line 1:33 no viable alternative at character '['" when visiting particular patch
		Restricted Task
Resolved	hashar	T103990 Gerrit upload-pack send ALL references causing massive network I/O on common operations
Resolved	None	T195280 New Gerrit UI does not contain copyable Change-Id or parent commit id
Open	None	T201246 Make a PolyGerrit plugin to welcome new users
Resolved	QChris	T254158 Gerrit 3.2 upgrade
Open	None	T256964 Gerrit commit message body is always wrapped with MS Edge

I'm lacking access to MS Edge. Could you please post a screenshot?

In T256964#6274601, @QChris wrote:

I'm lacking access to MS Edge. Could you please post a screenshot?

Sure.

The commit message:

And as it is shown with Gerrit:

If I understood CSS properly, the wrapping rule comes from the first rule in:

:host([pre]) span {
    white-space: var(--linked-text-white-space, pre-wrap);
    word-wrap: var(--linked-text-word-wrap, break-word);
}

And Microsoft Edge should support white-space: pre-wrap according to https://caniuse.com/#feat=mdn-css_properties_white-space_pre-wrap , and it supports var() as well. We would need to look at the dom/css rule applied to the element.