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

[Bug] [a11y] Delete Tab icon from collections is too small #4434

Closed
AndiAJ opened this issue Aug 1, 2019 · 5 comments
Closed

[Bug] [a11y] Delete Tab icon from collections is too small #4434

AndiAJ opened this issue Aug 1, 2019 · 5 comments
Assignees
Labels
access Accessibility: Talkback, HW keyboard/mouse, braile display etc. 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified S1 Blocks development/testing, may impact more than 25% of users, causes data loss, potential chemspill
Milestone

Comments

@AndiAJ
Copy link
Collaborator

AndiAJ commented Aug 1, 2019

Prerequisites

You have a collection with a couple of tabs saved.
You have Talkback option enabled from your Android settings.

Steps to reproduce

  1. Expand the collection
  2. Tap the 'x'(close) sign for a tab saved in that collection

Expected behavior

The sign can be easily tapped

Actual behavior

Difficult to tap it because of its size, could make an improvement as per #4221.

Device information

  • Android device: Google Pixel 3XL (Android 9)
  • Fenix version: RC 1.2.0-rc.1 from 1/8

┆Issue is synchronized with this Jira Task

@AndiAJ AndiAJ added the 🐞 bug Crashes, Something isn't working, .. label Aug 1, 2019
mcarare added a commit to mcarare/fenix that referenced this issue Aug 1, 2019
…cessible.

set recommended minimum size for accessibility 48x48, while keeping image size the same
removed margin from button as it was not needed anymore
aligned close button in center of tab to be visual consistent with alignment of favicon and more visual accessible
implemented same visual solution as for mozilla-mobile#4193 - close button for tabs
@AndiAJ AndiAJ added access Accessibility: Talkback, HW keyboard/mouse, braile display etc. S1 Blocks development/testing, may impact more than 25% of users, causes data loss, potential chemspill labels Aug 1, 2019
@mcarare mcarare self-assigned this Aug 1, 2019
@Djfe
Copy link

Djfe commented Aug 7, 2019

I agree, it's also not just the close button, but the webpage's icon, too and the text is cut off earlier due to padding (less characters for the maximum title length than an open tab), even though the overall size of the tab entry is the same.
644364412

mcarare added a commit to mcarare/fenix that referenced this issue Aug 12, 2019
… more accessible and aligned 3 dot menu with individual tabs close button

set buttons sizes to recommended minimum size for accessibility 48x48, while keeping image size the same
removed margins from buttons as they were not needed anymore
aligned center of menu buttons with center of collection icon
constrained description top to bottom of title, instead of share button to reduce empty space. (overlap with share button is already prevented by end constraint of description)
@sblatz sblatz closed this as completed in 1a4ef15 Aug 14, 2019
@sblatz sblatz added the eng:qa:needed QA Needed label Aug 14, 2019
@ekager ekager added this to the v1.3 milestone Aug 14, 2019
@AndiAJ AndiAJ added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Aug 16, 2019
@AndiAJ
Copy link
Collaborator Author

AndiAJ commented Aug 16, 2019

Hi, verified as fixed on 1.3.0-rc.1 using a Huawei Media Pad M3 Lite 10 (Android 7)

@lobontiumira
Copy link

Also verified as fixed on RC 1.3.0-rc.3 with Xiaomi Mi 8Lite (Android 9).

@Djfe
Copy link

Djfe commented Aug 19, 2019

should I open a new issue for the stuff I mentioned? because that's not fixed, yet.

@sblatz
Copy link
Contributor

sblatz commented Aug 19, 2019

@Djfe yes, please open a separate ticket if you're still having issues :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
access Accessibility: Talkback, HW keyboard/mouse, braile display etc. 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified S1 Blocks development/testing, may impact more than 25% of users, causes data loss, potential chemspill
Projects
None yet
Development

No branches or pull requests

6 participants