Open Bug 1550624 Opened 6 years ago Updated 3 years ago

Clean up Discovery uses of limit-visibile-lines

Categories

(Firefox :: New Tab Page, task, P3)

task

Tracking

()

People

(Reporter: Mardak, Unassigned)

References

Details

Attachments

(1 file)

Currently it's used to both set text sizing (font-size, line-height, etc) as well as controlling how many max lines to show (max-height and bug 1417163's line-clamp).

Separating the two would allow something that already has desired font size to be clamped to 1 line.

Oh, and might be good to rename the mixin at the same time (or at least not "visibile")

Depends on: 1417163
Assignee: nobody → gsuntop
Iteration: --- → 69.1 - May 13 - 26
Priority: -- → P1
Assignee: gsuntop → nobody
Priority: P1 → P3

Since we rolled back the dynamic line clamping this should be in our backlog.

Iteration: 69.1 - May 13 - 26 → 69.2 - May 27 - Jun 9
Iteration: 69.2 - May 27 - Jun 9 → ---
Component: Activity Streams: Newtab → New Tab Page
No longer blocks: pocket-newtab
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: