Bug 1669063 Comment 12 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

This is a WIP patch that doesn't work very well.
The main problem is that the CE header fields are populated a bit later and the full width size needed to check if the header is shrinking below that threshold is not correctly updated when the message becomes first visible.

Magnus, do you know if we have an observer that fires up when the message header is fully loaded with all data?

Another problem is the entire structure of this header, which is a bit complicated with many indented containers, most likely leftovers from de-xbl.
We should consider rebuilding this whole structure with pure HTML, CSS grid, and flexbox, which would make things easier to manage.

What do you think we should do here?
This is a WIP patch that doesn't work very well.
The main problem is that the CE header fields are populated a bit later and the full width size needed to check if the header is shrinking below that threshold is not correctly updated when the message becomes first visible.

Magnus, do you know if we have an observer that fires up when the message header is fully loaded with all data?

Another problem is the entire structure of this header, which is a bit complicated with many indented containers, most likely leftovers from de-xbl.
We should consider rebuilding this whole structure with pure HTML, CSS grid, and flexbox, which would make things easier to manage.

What do you think we should do here?

EDIT: Please ignore typos and crappy code in the patch. This is just a proof of concept.

Back to Bug 1669063 Comment 12