Closed Bug 666047 Opened 13 years ago Closed 13 years ago

GCLI completer text doesn't line up with the inputter text


(DevTools :: General, defect)

Not set


(Not tracked)



(Reporter: fitzgen, Unassigned)




(3 files, 1 obsolete file)

      No description provided.
Joe, this fixes the issue on my machine, but I am not sure it would work on all machines across the board. What would be a better way to fix this problem?
Attachment #540923 - Flags: feedback?(jwalker)
Just realized that this patch wrecks the existing jsterm's text alignment.
Attached file New patch
It seems that the reason the font alignment was messed up for GCLI but not jsterm is because jsterm uses a textbox element for the completer's node, but GCLI uses a div. I looked in to the styles that are applied to textboxes by default and it turns out that they have some margin by default!
Attachment #540923 - Attachment is obsolete: true
Attachment #540941 - Flags: feedback?(jwalker)
Attachment #540923 - Flags: feedback?(jwalker)
Comment on attachment 540941 [details]
New patch

It seems like a standard for CSS files to not have the tag name in there. i.e. s/div.jsterm/.jsterm/ and I know from previous work that the CSS files sometimes get significant scrutiny, so it's probably worth fixing.

We should (but to-date I have not) make similar changes in pinstripe/winstripe and gnomestripe. I'm guessing you're on a mac (pinstripe) so you won't easily be able to test the other configs anyway. I've vagely planned on doing that towards the end of the cycle (I've got a PC on order, which will help too)
Comment on attachment 540941 [details]
New patch

f+ the patch when the div part of the selector is gone.
Attachment #540941 - Flags: feedback?(jwalker) → feedback+
No longer blocks: 663120
Depends on: 666334
Can't happen until is completed because the div prefix is required b/c of the textbox vs. div default style issues that I mention above.
Closed: 13 years ago
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.