Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Overlapping text in upper-left pane (Cleopatra "Local Profile" list), due to leaking float effects #63

Open
dholbert opened this issue Aug 19, 2015 · 3 comments

Comments

@dholbert
Copy link

STR (for me at least):
(1) Record 4+ profiles using the Gecko Profiler Add-on.
(2) View one of them and look at your Local Profile list in upper-left

ACTUAL RESULTS: Overlapping stuff. See attached screenshot.
selection_005

@dholbert
Copy link
Author

The problem here is:
(1) The red "X" (fileListItemDelete) is a float, listed at the very end of each fileListItem.
(2) So, its static position is bleeding off the bottom of the entry. (You can see this if you disable position:relative.
(3) It is then relatively-positioned back up inside the entry -- but its layout effects lower down remain. (since relative positioning is just a shift-around-after-layout-has-happened kind of a thing)
(4) So, each fileListItem (each entry here) is really effectively taller than its specified size, because of these float bleed effects, and eventually it wraps around back into itself in a gross way.

@dholbert
Copy link
Author

I think the best fix would be to address "(1)" from my previous comment -- just move fileListItemDelete earlier in the DOM, e.g. put it between fileListItemTitle and fileListItemDescription.

Not sure where this has to happen -- presumably in whatever JS it is that generates this DOM.

@dholbert
Copy link
Author

Screencast showing my suggested fix in action (manually tweaking the DOM using devtools, and then adjusting the relative position "top" property of the red x's):
http://people.mozilla.org/~dholbert/bugreports/cleopatra-reorder-dom.ogv

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant