Limit label list height on edit issue page

[jaqra]

Before was no limit:

After:

[codecov-io]

Codecov Report

Merging #10216 into master will increase coverage by 0.03%.
The diff coverage is 50.41%.

@@            Coverage Diff             @@
##           master   #10216      +/-   ##
==========================================
+ Coverage   43.62%   43.66%   +0.03%     
==========================================
  Files         576      582       +6     
  Lines       79716    80045     +329     
==========================================
+ Hits        34778    34952     +174     
- Misses      40639    40757     +118     
- Partials     4299     4336      +37

Impacted Files	Coverage Δ
routers/org/home.go	65.68% <ø> (-0.34%)	⬇️
models/migrations/migrations.go	4.16% <ø> (ø)	⬆️
routers/user/profile.go	50.46% <ø> (-0.24%)	⬇️
routers/init.go	63.04% <0%> (-2.13%)	⬇️
modules/notification/indexer/indexer.go	50.66% <0%> (-4.41%)	⬇️
models/migrations/v127.go	0% <0%> (ø)
modules/graceful/manager_unix.go	40.83% <0%> (ø)	⬆️
modules/git/repo_commit.go	53.39% <0%> (ø)	⬆️
models/models.go	61.93% <100%> (+0.21%)	⬆️
modules/indexer/code/bleve.go	66.07% <100%> (ø)	⬆️
... and 24 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f34a4c...d68b6d3. Read the comment docs.

[jamesorlakin]

This also happens on the assignees on 1.11.0. I assume it's the same behaviour for labels but we don't have many... I think this should be backported, I'm happy to make a PR.

[jaqra]

as @proton-ab said i copied this rule from New Issue page

@lafriks marked as resolved. so i need to do anything?

@jamesorlakin ofcourse you can make a backport after merge :)

[guillep2k]

I think it's OK to maintain the 300px value for consistency, but perhaps we should change them all to a relative height in a future PR.

This one LGTM 👍

[jamesorlakin]

Having had a closer look while creating a backport, I'm not sure this will also fix a long assignments list as well. The extra CSS property is only assigned to the label selector, whereas the underlined selector is common between the two (_repository.less:91) and I feel should be changed instead.

(Names redacted because this is my work's 1.11 instance)

[jaqra]

@jamesorlakin can you send a pr?

[jamesorlakin]

Sure, I'll make a PR against master and then experiment with backporting it 👍