Page MenuHomePhabricator

Revisit the text size option to better communicate how to use it
Open, MediumPublic

Description

Problem

Users were able to identify and understand the text size feature. However, some of them struggled to understand how to use it when they started pressing the keys(4,5,6) while the text size window was still open, and it didn't change the text size. This ticket aims to communicate the best way to use the feature.

Proposed design

  • Article menu open up as a dialog box instead of a full screen.
  • Users can change and save the article text size preference using the slider.
  • A live view is available by changing the size of the dummy text and the article text in the background. Only change the dummy text when there is an image in the background.
  • Truncate the dummy text when users choose to increase the text size.
  • Preferred text size settings would be applicable only for the article title, body text, and information box.
  • Cancel softkey brings back the text size to previously saved size or default.
Article optionsOptions dialougeIncreased text sizeDecreased text size
zeplin:

https://zpl.io/VY3NWwd

Note: This change would be applicable for Jio and KaiOS users.

Event Timeline

Restricted Application added a project: Inuka-Team. · View Herald TranscriptNov 5 2020, 5:37 AM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
AMuigai moved this task from Backlog to Kanban on the Inuka-Team board.Nov 5 2020, 12:30 PM
AMuigai edited projects, added Inuka-Team (Kanban); removed Inuka-Team.
AMuigai triaged this task as Medium priority.Nov 5 2020, 12:41 PM

Here's 2 things that I think could help

  1. Add some text to explain that text size can be changed anywhere in the app
  2. Make it work really everywhere, including on this screen

Dropping ideas to discuss during daily.

AB
SGautam_WMF removed SGautam_WMF as the assignee of this task.Dec 1 2020, 5:56 PM
SGautam_WMF updated the task description. (Show Details)
SGautam_WMF moved this task from Design doing to Ready for Dev on the Inuka-Team (Kanban) board.
eamedina claimed this task.Dec 1 2020, 6:01 PM
eamedina moved this task from Ready for Dev to Dev on the Inuka-Team (Kanban) board.
SGautam_WMF updated the task description. (Show Details)Dec 2 2020, 5:06 AM

Updating ticket description to accommodate for T269233, as discussed in our daily meeting today

eamedina updated the task description. (Show Details)Dec 2 2020, 8:36 PM

Here's the PR and branch name: https://github.com/wikimedia/wikipedia-kaios/pull/290 - T267293-text-size

@SGautam_WMF would love to sync with you after you have a chance to play with it, let's talk about the LSK

eamedina moved this task from Dev to Code Review on the Inuka-Team (Kanban) board.Dec 7 2020, 12:29 PM
eamedina added a comment.EditedDec 10 2020, 2:34 PM

This ticket has now become a multi-PR ticket: we have gone ahead and merged https://github.com/wikimedia/wikipedia-kaios/pull/290 and I will continue working on some fine-tuning in a follow up PR. We wanted to get the strings translations rolling as soon as possible, hence the decision to go with more than one PR.

As of right now, this is what I got for the upcoming fine-tuning:

  • We have spotted a code refactor opportunity we want to take advantage of
  • Fine-tune the sizing and styling, make sure it looks fine in Jio2 devices
  • Backspace key fine-tuning
  • Visibility of the popup behind

Will bring this ticket back to dev for the time being, stay tuned

eamedina moved this task from Code Review to Dev on the Inuka-Team (Kanban) board.Dec 10 2020, 2:34 PM

Fine tuning PR is https://github.com/wikimedia/wikipedia-kaios/pull/296, this addresses the priorities for release: styling for Jio2 devices and back popup visibility.

PR for refactor and the rest coming next.

eamedina moved this task from Dev to Code Review on the Inuka-Team (Kanban) board.Mon, Jan 11, 2:11 PM
SBisson moved this task from Code Review to QA on the Inuka-Team (Kanban) board.Mon, Jan 18, 7:09 PM
SBisson added a subscriber: Jpita.

The big refactoring PR is merged. @Jpita please test carefully.