Verified
Status Update
Comments
sv...@axis.com <sv...@axis.com>
sv...@axis.com <sv...@axis.com>
ap...@google.com <ap...@google.com> #2
Ok, here is a fix:
422524: Fix vertical alignment of header icon and text |
I have spent enough time to fully understand what is going on, so I could as well fix the issue. :-) What convinced me is that even all the current installations (e.g. chromium-review) have improved. So this was not just a feature request for using other icon sizes, but this was an alignment bug everywhere, even though it mostly matters just 1 or 2 pixels.
sv...@axis.com <sv...@axis.com>
sv...@axis.com <sv...@axis.com> #4
Project: gerrit
Branch: master
commit 06cfcb6ef54d78d7e68dfc0f17a15d30d6aae414
Author: Ben Rohlfs <brohlfs@google.com>
Date: Fri Apr 26 11:28:12 2024
Fix vertical alignment of header icon and text
There is generally a slight misalignment of icon and text as can be
seen in this screenshot:https://imgur.com/a/tDUHGk0
It was also noted in the referenced bug that increasing the icon height
is impossible, because then the alignment gets totally messed up.
3 problems contribute to this misalignment:
1. `.titleText` element being a span. This generally makes alignment of
icons next to text very tricky. Let's use a flex element instead.
2. `vertical-align: text-bottom` for the icon is a very unusual choice
for the vertical-align property and does not really make a lot of
sense. Let's just use the `align-items: center` style for flex
elements, see above.
3. `.bigTitle` has a custom font size, but not a custom line-height.
That is generally problematic as can be seen here: The element will
get a height of 20px as an inline element, but contains text with a
24.5px font. No surprise that we see misalignment. Let's make sure
that the line-height and thus the height of the text element has a
proper value. 1.2*font-size is what browsers would normally use.
This was checked to behave well on gerrit-review, android-review and
chromium-review.
Release-Notes: skip
Bug: Issue 337076005
Change-Id: Ia09bbe92158f7b4b1d6b88e599e0635fa9b15d5e
M polygerrit-ui/app/elements/core/gr-main-header/gr-main-header.ts
M polygerrit-ui/app/elements/core/gr-main-header/gr-main-header_test.ts
https://gerrit-review.googlesource.com/422524
Branch: master
commit 06cfcb6ef54d78d7e68dfc0f17a15d30d6aae414
Author: Ben Rohlfs <brohlfs@google.com>
Date: Fri Apr 26 11:28:12 2024
Fix vertical alignment of header icon and text
There is generally a slight misalignment of icon and text as can be
seen in this screenshot:
It was also noted in the referenced bug that increasing the icon height
is impossible, because then the alignment gets totally messed up.
3 problems contribute to this misalignment:
1. `.titleText` element being a span. This generally makes alignment of
icons next to text very tricky. Let's use a flex element instead.
2. `vertical-align: text-bottom` for the icon is a very unusual choice
for the vertical-align property and does not really make a lot of
sense. Let's just use the `align-items: center` style for flex
elements, see above.
3. `.bigTitle` has a custom font size, but not a custom line-height.
That is generally problematic as can be seen here: The element will
get a height of 20px as an inline element, but contains text with a
24.5px font. No surprise that we see misalignment. Let's make sure
that the line-height and thus the height of the text element has a
proper value. 1.2*font-size is what browsers would normally use.
This was checked to behave well on gerrit-review, android-review and
chromium-review.
Release-Notes: skip
Bug:
Change-Id: Ia09bbe92158f7b4b1d6b88e599e0635fa9b15d5e
M polygerrit-ui/app/elements/core/gr-main-header/gr-main-header.ts
M polygerrit-ui/app/elements/core/gr-main-header/gr-main-header_test.ts
sv...@axis.com <sv...@axis.com> #5
Project: gerrit
Branch: stable-3.8
commit 518840c5b86419c19e4fa2616e273a25998fad9b
Author: Ben Rohlfs <brohlfs@google.com>
Date: Fri Apr 26 11:28:12 2024
Fix vertical alignment of header icon and text
There is generally a slight misalignment of icon and text as can be
seen in this screenshot:https://imgur.com/a/tDUHGk0
It was also noted in the referenced bug that increasing the icon height
is impossible, because then the alignment gets totally messed up.
3 problems contribute to this misalignment:
1. `.titleText` element being a span. This generally makes alignment of
icons next to text very tricky. Let's use a flex element instead.
2. `vertical-align: text-bottom` for the icon is a very unusual choice
for the vertical-align property and does not really make a lot of
sense. Let's just use the `align-items: center` style for flex
elements, see above.
3. `.bigTitle` has a custom font size, but not a custom line-height.
That is generally problematic as can be seen here: The element will
get a height of 20px as an inline element, but contains text with a
24.5px font. No surprise that we see misalignment. Let's make sure
that the line-height and thus the height of the text element has a
proper value. 1.2*font-size is what browsers would normally use.
This was checked to behave well on gerrit-review, android-review and
chromium-review.
Release-Notes: skip
Bug: Issue 337076005
Change-Id: Ia09bbe92158f7b4b1d6b88e599e0635fa9b15d5e
(cherry picked from commit 06cfcb6ef54d78d7e68dfc0f17a15d30d6aae414)
M polygerrit-ui/app/elements/core/gr-main-header/gr-main-header.ts
M polygerrit-ui/app/elements/core/gr-main-header/gr-main-header_test.ts
https://gerrit-review.googlesource.com/425217
Branch: stable-3.8
commit 518840c5b86419c19e4fa2616e273a25998fad9b
Author: Ben Rohlfs <brohlfs@google.com>
Date: Fri Apr 26 11:28:12 2024
Fix vertical alignment of header icon and text
There is generally a slight misalignment of icon and text as can be
seen in this screenshot:
It was also noted in the referenced bug that increasing the icon height
is impossible, because then the alignment gets totally messed up.
3 problems contribute to this misalignment:
1. `.titleText` element being a span. This generally makes alignment of
icons next to text very tricky. Let's use a flex element instead.
2. `vertical-align: text-bottom` for the icon is a very unusual choice
for the vertical-align property and does not really make a lot of
sense. Let's just use the `align-items: center` style for flex
elements, see above.
3. `.bigTitle` has a custom font size, but not a custom line-height.
That is generally problematic as can be seen here: The element will
get a height of 20px as an inline element, but contains text with a
24.5px font. No surprise that we see misalignment. Let's make sure
that the line-height and thus the height of the text element has a
proper value. 1.2*font-size is what browsers would normally use.
This was checked to behave well on gerrit-review, android-review and
chromium-review.
Release-Notes: skip
Bug:
Change-Id: Ia09bbe92158f7b4b1d6b88e599e0635fa9b15d5e
(cherry picked from commit 06cfcb6ef54d78d7e68dfc0f17a15d30d6aae414)
M polygerrit-ui/app/elements/core/gr-main-header/gr-main-header.ts
M polygerrit-ui/app/elements/core/gr-main-header/gr-main-header_test.ts
ap...@google.com <ap...@google.com> #6
Project: gerrit
Branch: stable-3.8
commit 4312d4d00ad3e6d2cf261d7b670c27bea5bf7c51
Author: David Åkerman <davidak@axis.com>
Date: Wed Jun 05 10:50:22 2024
Match hashtags suggestions with regex as well
In previous change 428377 we missed to handle the case
where the input is a regex. This will fix that.
Solves: bug 344608194
Release-Notes: skip
Change-Id: I8f1588c09459c76e888b22b7f992b9a8f0064c99
M polygerrit-ui/app/elements/change/gr-change-metadata/gr-change-metadata.ts
https://gerrit-review.googlesource.com/428559
Branch: stable-3.8
commit 4312d4d00ad3e6d2cf261d7b670c27bea5bf7c51
Author: David Åkerman <davidak@axis.com>
Date: Wed Jun 05 10:50:22 2024
Match hashtags suggestions with regex as well
In previous change 428377 we missed to handle the case
where the input is a regex. This will fix that.
Solves:
Release-Notes: skip
Change-Id: I8f1588c09459c76e888b22b7f992b9a8f0064c99
M polygerrit-ui/app/elements/change/gr-change-metadata/gr-change-metadata.ts
Description
What steps will reproduce the problem?
1. Create change and give it three hastags ["lemon", "apple", "kiwi"]
2. Create another change and enter "lem" in the hashtag input.
3.
What is the expected output?
The suggest should be:
* lemon
What do you see instead?
Suggest is:
* lemon
* apple
* kiwi
What is the output of the JS console log (if applicable)?
What is the performance record (see
Please provide any additional information below.
The hashtag suggest currently looks for changes with similar hashtags and uses ALL hashtags from those changes regardless of whether they match the input or not.
Functionality is identical to the topic suggest but since there can only be 1 topic / change the topic suggest works perfect.