Closed Bug 1050356 Opened 10 years ago Closed 10 years ago

Make search widget indentation consistent with FirefoxSearch repo

Categories

(Firefox for Android Graveyard :: Search Activity, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 34

People

(Reporter: Margaret, Assigned: Margaret)

References

Details

Attachments

(1 file, 1 obsolete file)

Looks like the m-c patch that landed for bug 1046419 was different from the git commit. This fixes the issue with another patch for m-c, but we could also make a fixup commit to the git repo if you think what's in m-c is better.

We need to do something to fix this, though, since it breaks generating patches from the FirefoxSearch repo.
Attachment #8469364 - Flags: review?(wjohnston)
Comment on attachment 8469364 [details] [diff] [review]
Make search widget indentation consistent with FirefoxSearch repo

Review of attachment 8469364 [details] [diff] [review]:
-----------------------------------------------------------------

Heh. Sorry about that.
Attachment #8469364 - Flags: review?(wjohnston) → review+
Assignee: nobody → margaret.leibovic
Attached patch Patch (obsolete) — Splinter Review
I think the remaining problems are mostly due to fonts. We have to do some weird things here so that the gray background doesn't overlap the orange (hence why these rows were offset before).
Comment on attachment 8470279 [details] [diff] [review]
Patch

Wrong bug :(
Attachment #8470279 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/61b29b65287f
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: