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

Allow to change the home screen background by tapping on the firefox logo #22851

Closed
Amejia481 opened this issue Dec 14, 2021 · 5 comments
Closed
Assignees
Labels
eng:qa:verified QA Verified feature request 🌟 New functionality and improvements Feature:Wallpapers needs:triage Issue needs triage

Comments

@Amejia481
Copy link
Contributor

Amejia481 commented Dec 14, 2021

┆Issue is synchronized with this Jira Task

@Amejia481 Amejia481 added feature request 🌟 New functionality and improvements Feature:Wallpapers labels Dec 14, 2021
@Amejia481 Amejia481 self-assigned this Dec 14, 2021
@github-actions github-actions bot added the needs:triage Issue needs triage label Dec 14, 2021
Amejia481 added a commit to Amejia481/fenix that referenced this issue Dec 14, 2021
mergify bot pushed a commit that referenced this issue Dec 14, 2021
@Amejia481 Amejia481 changed the title Allow to change the home screen background by double tap on the firefox logo Allow to change the home screen background by tapping on the firefox logo Dec 14, 2021
@sheikh-azharuddin
Copy link

Nice implementation 👍
These 2 backgrounds color have issue with the text color ...

Screenshot_20211220-231215104
Screenshot_20211220-231234021

@pocmo
Copy link
Contributor

pocmo commented Dec 21, 2021

@Amejia481 Two suggestions:

  • I found the commit and this issue by looking at the APK size graphs and saw a bump, which seems to be coming from the new wallpaper PNGs. Did you run them through ImageOptim? It looks like we could optimize them.
  • I'd consider adding such state either to the screen or app store (based on whether it's needed locally only or globally) and using the known mechanisms to update/observe state. Every *Manager creates another source of truth/state and will make it harder to reason about the app's state.

@Amejia481
Copy link
Contributor Author

Amejia481 commented Dec 21, 2021

👋🏽 thanks for the suggestions.

Did you run them through ImageOptim?

No I didn't, but I will do :) . These images are the final ones, they are only for gathering feedback about the feature, while we get the final ones.

I'd consider adding such state either to the screen or app store (based on whether it's needed locally only or globally) and using the known mechanisms to update/observe state. Every *Manager creates another source of truth/state and will make it harder to reason about the app's state.

Yeah that make sense 👍🏽

Amejia481 added a commit to Amejia481/fenix that referenced this issue Dec 21, 2021
@Amejia481
Copy link
Contributor Author

@pocmo I filed #22937 for addressing the images optimization.

Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 13, 2022
mergify bot pushed a commit that referenced this issue Jan 13, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 18, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 19, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 19, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 19, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 19, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 19, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 19, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 21, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 24, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 24, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 24, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 26, 2022
Amejia481 added a commit to Amejia481/fenix that referenced this issue Jan 26, 2022
mergify bot pushed a commit that referenced this issue Jan 27, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 8, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 8, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 8, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 8, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 9, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 9, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 9, 2022
pkirakosyan pushed a commit to gexsi/user-agent-android that referenced this issue Mar 9, 2022
@delia-pop
Copy link

Verified as fixed on Beta 99.0.0-beta.5, Nightly 100.0a1. and RC 98.3.0.

@delia-pop delia-pop added the eng:qa:verified QA Verified label Mar 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:qa:verified QA Verified feature request 🌟 New functionality and improvements Feature:Wallpapers needs:triage Issue needs triage
Projects
None yet
Development

No branches or pull requests

4 participants