Closed Bug 1262489 Opened 8 years ago Closed 8 years ago

'Show web fonts' setting could still be improved

Categories

(Firefox for Android Graveyard :: Settings and Preferences, defect)

defect
Not set
normal

Tracking

(firefox48 verified, fennec48+)

RESOLVED FIXED
Firefox 48
Tracking Status
firefox48 --- verified
fennec 48+ ---

People

(Reporter: mfinkle, Assigned: Margaret)

References

Details

Attachments

(1 file)

I still have some issues with the 'Show web fonts' setting:
1. The subtitle is confusing
2. Using a checkbox limits our ability to convey context and breaks the visual 'grouping' with 'Show images'

#1 "Hide to use default fonts and reduce website load times"
The "Hide to" part of the text makes it harder to parse and understand. If we keep this as a checkbox, I think we should consider dropping the "Hide to" part.

#2 Use of a checkbox
Using a list, like 'Show images', gives us more freedom to give context about the 'on' and 'off' states of the setting. We obviously felt the setting needed more context. Using a list fits in better visually with 'Show images' too. Even if we only have two states, using text to convey the context of both states (list) is better than describing only one of the states (checkbox). We also have plans to add a new state to the setting in bug 1250989, so let's plan ahead.
tracking-fennec: --- → ?
(In reply to Mark Finkle (:mfinkle) from comment #0)
> I still have some issues with the 'Show web fonts' setting:
> 1. The subtitle is confusing
> 2. Using a checkbox limits our ability to convey context and breaks the
> visual 'grouping' with 'Show images'
> 
> #1 "Hide to use default fonts and reduce website load times"
> The "Hide to" part of the text makes it harder to parse and understand. If
> we keep this as a checkbox, I think we should consider dropping the "Hide
> to" part.

I agree, this could be better. Let's try to come up with a better string.

> #2 Use of a checkbox
> Using a list, like 'Show images', gives us more freedom to give context
> about the 'on' and 'off' states of the setting. We obviously felt the
> setting needed more context. Using a list fits in better visually with 'Show
> images' too. Even if we only have two states, using text to convey the
> context of both states (list) is better than describing only one of the
> states (checkbox). We also have plans to add a new state to the setting in
> bug 1250989, so let's plan ahead.

While a dialog for only on/off states seems like a bit of overkill, I agree in the broader context of how this setting is presented, it would make things more consistent.
Assignee: nobody → margaret.leibovic
tracking-fennec: ? → 48+
Flags: needinfo?(alam)
Yeah, this string is really confusing. Especially since we place it next to a "control" that has an on/off state.

Also, do we need to capitalize "web" here?

How about:

Show web fonts
Download typefaces when you load a page

We are looking to take a complete look at all of this from a content point of view in the near future. So let's keep that in mind too and stay in scope :)
Flags: needinfo?(alam)
(In reply to Barbara Bermes [:barbara] from comment #8)
> Sorry for the delay; I'm probably being a bit too stingy about this but as a
> user I might get confused about the check/uncheck box behaviour as it
> relates to the text. 
> 
> We need to make sure that the description is not contradicting with the
> title. 
> 
> "Show xyz", and the description carries the word "hide/disable", feels
> confusing. I know we use a positive word such as "show" for the item above,
> "show images", but maybe we should use a negative word for the web fonts.
> 
> Thoughts about the following:
> 
> Title: "Disable Web fonts"
> Description: "Use default fonts and reduce website load times"
Flags: needinfo?(matej)
Flags: needinfo?(krudnitski)
Flags: needinfo?(krudnitski)
(In reply to :Margaret Leibovic from comment #3)
> (In reply to Barbara Bermes [:barbara] from comment #8)
> > Sorry for the delay; I'm probably being a bit too stingy about this but as a
> > user I might get confused about the check/uncheck box behaviour as it
> > relates to the text. 
> > 
> > We need to make sure that the description is not contradicting with the
> > title. 
> > 
> > "Show xyz", and the description carries the word "hide/disable", feels
> > confusing. I know we use a positive word such as "show" for the item above,
> > "show images", but maybe we should use a negative word for the web fonts.
> > 
> > Thoughts about the following:
> > 
> > Title: "Disable Web fonts"
> > Description: "Use default fonts and reduce website load times"

Maybe we could say ""Use default fonts to reduce website load times" so it's clear there's a relationship between the two. Otherwise, looks great to me.
Flags: needinfo?(matej)
For me, if we were to move away from the check box (since that's the only check box in this menu) and towards one of the other models (like 'show images' or 'plugins' or 'character encoding'), I think we have a better chance for clarity.

ie if the Title is as Barbara suggested: "Disable Web fonts", we can have two choices:
"Use default fonts to reduce page load times"
"Download Web fonts requested by page"

or something along those lines. Basically, tie the explanation to the binary choice description.
Final suggestion here, collecting all ideas:

Replace Checkbox with "show images" like-list

Title: Hide Web fonts
Description: Use default fonts to reduce page load times
Values: Always | Only without Wi-Fi | Never

The "Show" wording does not give enough credit to what we are trying to explain to the user: similar to images, we want to offer the user choices to reduce their page load time and data. 

If we, however, would use "Disable" for Images _and_ Web Fonts, we would have a better chance to show the value in this. What do you think?
Flags: needinfo?(matej)
Flags: needinfo?(margaret.leibovic)
Flags: needinfo?(alam)
(In reply to Barbara Bermes [:barbara] from comment #6)
> Final suggestion here, collecting all ideas:
> 
> Replace Checkbox with "show images" like-list
> 
> Title: Hide Web fonts
> Description: Use default fonts to reduce page load times
> Values: Always | Only without Wi-Fi | Never

I think this does a good job of communicating the benefit, but at the same time, the wording is a bit confusing. "Hide," while active as a verb, suggests something that isn't going to happen, so pairing it with "Never" takes a moment to figure out (i.e. "never don't do this thing"). "Without Wi-Fi" is another negative state.

Could we put "Use default fonts" in the title to put everything in the positive? That would look something like:

Title: Use default fonts
Description: Hide Web fonts to reduce page load times
Values: Always | Only without Wi-Fi | Never

I still wish the Wi-Fi one could be worded differently, but the only thing I can come up with is something like "Only on a data connection" and that's pretty long.

WDYT?
Flags: needinfo?(matej)
(In reply to Barbara Bermes [:barbara] from comment #6)
> Final suggestion here, collecting all ideas:
> 
> Replace Checkbox with "show images" like-list
> 
> Title: Hide Web fonts
> Description: Use default fonts to reduce page load times
> Values: Always | Only without Wi-Fi | Never

Hold up here... we don't actually have the ability to show fonts "only on wi-fi" (bug 1262489).

For a patch here, we could only support "Always" and "Never"

And if we're using a dialog, like "show images", there's no room for a description, so it would appear like this to users:

Hide Web fonts
Always

We need a solution that's technically possible. The one proposed is not.
Flags: needinfo?(margaret.leibovic) → needinfo?(bbermes)
Ok, antlam, margaret and myself decided the following:

This bug (https://bugzilla.mozilla.org/show_bug.cgi?id=1261137) is a prerequisite for the following:

1)
Replace checkbox with switch
2) 
Title: Show web fonts
Description: Download remote fonts when loading a page (we took this from the wikipedia explanation)

"Show images" and "Show web fonts" will be under "Data Saver" grouping to address the value prop of saving data to the user.
Flags: needinfo?(bbermes)
Flags: needinfo?(alam)
Comment on attachment 8740660 [details]
MozReview Request: Bug 1262489 - Improve wording of "Show web fonts" setting. r=sebastian

https://reviewboard.mozilla.org/r/45863/#review42569
Attachment #8740660 - Flags: review?(s.kaspari) → review+
https://hg.mozilla.org/integration/fx-team/rev/1578157460d3dad2a0585ef96f7c84a3f04d31a3
Bug 1262489 - Improve wording of "Show web fonts" setting. r=sebastian
https://hg.mozilla.org/mozilla-central/rev/1578157460d3
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
- Checkbox is replaced with switch
-"Show web fonts": Download remote fonts when loading a page
-"Show web fonts" is displayed along "Show images" in "Data Saver" sub-group

Verified as fixed using:
Device: Nexus 6 (Android 6.0)
Build: Firefox for Android 48.0a1 (2016-04-17)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.