Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Update UX for history management #563

Closed
Tracked by #177
TitanNano opened this issue Feb 17, 2019 · 16 comments
Closed
Tracked by #177

Update UX for history management #563

TitanNano opened this issue Feb 17, 2019 · 16 comments
Assignees
Labels
eng:qa:verified QA Verified eng:ready Ready for engineering Feature:History P1 Current sprint

Comments

@TitanNano
Copy link
Contributor

TitanNano commented Feb 17, 2019

Steps to reproduce

Go into Fenix history management

Expected behavior

On android list actions are usually in the page header. When a list item is selected the header switches into edit mode, providing special editing actions.

Actual behavior

The current history management is very unandroidy. There is a delete-history-button right below the header which changes into "delete x Items" when a history entry is selected. I attached a screenshot of how Google Messages does it. The X on the right side of the header is also very strange.

Device information

  • Android device: 7.1.1
  • Fenix version: 1.0

screenshot_20190217-151747
screenshot_20190217-145929

┆Issue is synchronized with this Jira Task

@yoasif
Copy link
Contributor

yoasif commented Feb 17, 2019

FWIW, Bromite/Kiwi have a more "Android" UI around history management that replicates Fenix's features in a nicer way.

screenshot feb 17 2019 1_30_43 pm

A couple of things:

  • No three dot menu in Bromite; as the only option is delete, they use an icon for it
  • They also have the "X" in the header to exit this view

screenshot feb 17 2019 1_35_29 pm

The selection of multiple items is just as undiscoverable as on Fenix, (hold on favicon) but that seems to be an Android-ism.

As the reporter noted, once selected, the header changes to show options to perform with the selected items; interestingly, the options inside the three dot menu include opening the history entries in new tabs (either normal or private). The header text also updates to show the number of selected items.

@vesta0 vesta0 added the needs:UX-feedback Needs UX Feedback label Feb 19, 2019
@bbinto
Copy link
Contributor

bbinto commented Feb 19, 2019

Some of these issue have been discussed with UX, will review once the Library UI is done

@bbinto bbinto added this to the Milestone 3 Backlog milestone Feb 19, 2019
@lime124 lime124 mentioned this issue Mar 13, 2019
2 tasks
@topotropic topotropic self-assigned this Mar 13, 2019
@vesta0 vesta0 added the P1 Current sprint label Apr 1, 2019
@vesta0 vesta0 modified the milestones: MVP Backlog, Bugs Apr 1, 2019
@vesta0 vesta0 added the UX label Apr 4, 2019
@topotropic
Copy link

@boek could you update history so that selections matches the spec here https://mozilla.invisionapp.com/share/8KRF5PW9Q6Z ?

@vesta0 vesta0 removed the needs:UX-feedback Needs UX Feedback label Apr 23, 2019
@vesta0 vesta0 changed the title Unusual UX for history management Update UX for history management Apr 30, 2019
@vesta0 vesta0 modified the milestones: Bugs, MVP Backlog Apr 30, 2019
@topotropic
Copy link

@vesta0 @bifleming Is this still blocked?

@vesta0
Copy link
Collaborator

vesta0 commented Apr 30, 2019

@topotropic no I think it's ready for engineering.

@vesta0 vesta0 added the eng:ready Ready for engineering label Apr 30, 2019
@topotropic topotropic removed the UX label Apr 30, 2019
@topotropic topotropic removed their assignment May 6, 2019
@sblatz sblatz self-assigned this May 7, 2019
sblatz added a commit to sblatz/fenix that referenced this issue May 10, 2019
sblatz added a commit to sblatz/fenix that referenced this issue May 10, 2019
@sblatz sblatz added eng:qa:needed QA Needed and removed in progress eng:qa:needed QA Needed labels May 10, 2019
@bsurd
Copy link

bsurd commented May 13, 2019

The checkmark when selecting history elements does not match the one from the design specs in invision. Bookmark items are displayed as the design specs.
screen

There are also inconsistencies between bookmarks and history elements. The bookmarks are displayed with a circle and history ones have a square that turns into a circle when long tapping to select multiple elements.
screen

@sblatz Could you take a look at this?

@bsurd bsurd removed the eng:qa:needed QA Needed label May 13, 2019
@cadeyrn
Copy link
Contributor

cadeyrn commented May 13, 2019

See also #2435.

sblatz pushed a commit to sblatz/fenix that referenced this issue May 13, 2019
sblatz added a commit to sblatz/fenix that referenced this issue May 13, 2019
@sblatz
Copy link
Contributor

sblatz commented May 14, 2019

@topotropic for the fav icons in bookmarks and history, do we want them to be the rounded square or the square inside the circle?

@topotropic
Copy link

@sblatz circles

@sblatz sblatz added the eng:qa:needed QA Needed label May 14, 2019
@cadeyrn
Copy link
Contributor

cadeyrn commented May 14, 2019

@topotropic Then the favicons in the "open tabs" section on the home screen should also be changed because it doesn't make much sense to have different favicon styles across the product. Now just one inconsistency has been swapped for another inconsistency.

By the way, in my opinion it looks much cleaner without the circle and the design without the circle has also the advantage of bigger icons.

@yoasif
Copy link
Contributor

yoasif commented May 14, 2019

FWIW, Bromite, Kiwi, Opera all use circles for bookmarks on Android. Edge does not, but it also doesn't seem to care about looking like an Android app.

I think #2132 is a bigger issue as compared to other Android apps, but I don't have much of an opinion about the circles.

@cadeyrn
Copy link
Contributor

cadeyrn commented May 14, 2019

I've never heard of Kiwi and Bromite but in Fennec there are also no circles. ;-) But it's a design decision and therefore not important what others do. Fenix as a whole should look coherent. So I can live with both styles but it could be better to use only one style for all places. ;-)

@yoasif
Copy link
Contributor

yoasif commented May 14, 2019

Sad. I forgot to check Fennec, and I like the Fennec design the most, upon review.

@topotropic
Copy link

Thanks for your feedback! I can see where you are coming from. The idea behind using squares on start screen and circles on history/bookmarks is that the context is a different one: on start screen images are embeded in cards, on history/bookmarks we use list elements. The placeholders on start screen are also slightly bigger and this format allows room for using other source material then favicons, for example images. Hope that helps.

@bsurd
Copy link

bsurd commented May 15, 2019

Verified on the latest build (5/15) on a Nexus 5 and OnePlus 5T.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:qa:verified QA Verified eng:ready Ready for engineering Feature:History P1 Current sprint
Projects
None yet
Development

No branches or pull requests

10 participants