Closed Bug 1066033 Opened 5 years ago Closed 5 years ago

Use orange text selection handles in search bar

Categories

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

All
Android
defect
Not set

Tracking

(firefox36 verified)

VERIFIED FIXED
Firefox 35
Tracking Status
firefox36 --- verified

People

(Reporter: Margaret, Assigned: Margaret)

References

Details

Attachments

(1 file)

A bit of polish for the search activity.

It would also be nice to use orange as the highlight color for the action bar, but I'm not sure how to do that.
Wes, do you know how I can change the color of the action bar highlight as well?

Also, I just made copies of the xhdpi resources in fennec-search so that it will still build, but `grunt export` won't export them since they already exist in mozilla-central.
Attachment #8487877 - Flags: review?(wjohnston)
Comment on attachment 8487877 [details] [diff] [review]
Use orange text selection handles in search bar

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

I assume the highlight is http://developer.android.com/reference/android/widget/TextView.html#attr_android:textColorHighlight

We set that in our theme for Fennec, so it applies to everything.
Attachment #8487877 - Flags: review?(wjohnston) → review+
(In reply to Wesley Johnston (:wesj) from comment #2)
> Comment on attachment 8487877 [details] [diff] [review]
> Use orange text selection handles in search bar
> 
> Review of attachment 8487877 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I assume the highlight is
> http://developer.android.com/reference/android/widget/TextView.
> html#attr_android:textColorHighlight
> 
> We set that in our theme for Fennec, so it applies to everything.

Are you suggesting I should remove that from this TextView? I found I needed to add that myself at one point, but perhaps that was for the standalone search activity.
https://hg.mozilla.org/mozilla-central/rev/69d5ff91f0aa
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 35
Text selection handles in search results are blue. Is this as expected?
(In reply to Flaviu Cos, QA [:flaviu] from comment #6)
> Text selection handles in search results are blue. Is this as expected?

Yes, this is because this is a WebView, so we're just using the defaults that are there. Perhaps it could be worth filing a separate bug to update this, but I don't know that it's really important, since we may eventually move away from the WebView.
Verified as fixed in build 36.0a1 2014-11-17;
Device: Asus Transformer Tab (Android 4.0.3).
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.