Closed Bug 821311 Opened 8 years ago Closed 8 years ago

PBM - Search suggestion text illegible

Categories

(Firefox for Android :: Theme and Visual Design, defect)

20 Branch
ARM
Android
defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 20
Tracking Status
firefox20 --- verified

People

(Reporter: aaronmt, Assigned: sriram)

References

Details

Attachments

(2 files)

Not sure if another theming bug already will cover this or not; see screenshot.

--
Nightly (12/13)
Sriram - Is this a dupe of the Persona text bug, or is it different?
Assignee: nobody → sriram
Attached patch PatchSplinter Review
Changed the TextView to GeckoTextView and set the private mode when inflated.
Attachment #692053 - Flags: review?(bnicholson)
Comment on attachment 692053 [details] [diff] [review]
Patch

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

::: mobile/android/base/awesomebar/AllPagesTab.java
@@ +602,5 @@
> +        GeckoTextView promptText = (GeckoTextView) mSuggestionsOptInPrompt.findViewById(R.id.suggestions_prompt_title);
> +        promptText.setText(getResources().getString(R.string.suggestions_prompt, mSearchEngines.get(0).name));
> +
> +        Tab tab = Tabs.getInstance().getSelectedTab();
> +        if (tab != null && tab.isPrivate())

I hate these superfluous null checks...we should maybe try a Try push without them since Talos no longer does session restores (which I believe is why you needed to add these checks to begin with).
Attachment #692053 - Flags: review?(bnicholson) → review+
https://hg.mozilla.org/mozilla-central/rev/e1a57065f541
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
Firefox 20.0a1 (2012-11-19)
Device: Galaxy Nexus
OS: Android 4.1.1 

This is not reproducible on latest Nightly. Marking bug as Verified Fixed.
Status: RESOLVED → VERIFIED
Depends on: 826267
You need to log in before you can comment on or make changes to this bug.