1621160 - First row of stagger column headers will be hidden when table header becomes sticky

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Search, defect)

Description

[Daniel Veditz [:dveditz]]

I added the "regressed by" column to my buglists. The data shown in that column is actually the Duplicates data. Filing against BMO rather than generic Bugzilla buglists because I think "regressed by" is a local customization. Maybe a copy/paste error, and if so might be worth checking the similar columns created at the same time.

Comment 1

[Kohei Yoshino [:kohei]]

Hmm, interesting. I implemented the new columns in Bug 1461492, and it seems working for me.

e.g. https://bugzilla.mozilla.org/buglist.cgi?chfield=%5BBug%20creation%5D&chfieldfrom=-1m&columnlist=short_desc,regressed_by,regresses,duplicates&keywords=regression

Changelog: https://github.com/mozilla-bteam/bmo/pull/1067/files

Somehow bug IDs in the Regressions and Regressed by columns are not linked, which is probably a bug.

Comment 2

[Daniel Veditz [:dveditz]]

D'oh! I have staggered columns, and when the list is scrolled half the column headers go away. My buglist apparently had zero regressions on it and what I was looking at was, in fact, the headerless duplicates column. No wonder what I got were the duplicates, then.

Do you have a bug filed on the fact that when there's more than one duplicate the buglinks don't work? They do for a single duplicate, but multiple gives urls like https://bugzilla.mozilla.org/show_bug.cgi?id=156461,%20175933,%20215659,%20244440,%20248008,%20280581 -- note "show_bug" vs. "buglist".

Comment 3

[Daniel Veditz [:dveditz]]

"regressions" and "regressed by" columns don't even try to be bug links. It would be nice if they were.

Comment 4

[Kohei Yoshino [:kohei]]

Do you have a bug filed on the fact that when there's more than one duplicate the buglinks don't work?

Ah, didn’t notice that. Why no one has ever filed a bug for that? ;-) Will fix it along with IDs in the regression columns.

Comment 5

[Kohei Yoshino [:kohei]]

And I’ll fix the stagger header issue as well. Actually I didn’t test it when I implemented the sticky table header...

Comment 6

[Kohei Yoshino [:kohei]]

Attachment: GitHub Pull Request

Comment 7

[David Lawrence [:dkl]]

Merged to master.