mozilla-mobile / fenix

⚠️ Fenix (Firefox for Android) moved to a new repository. It is now developed and maintained as part of: https://github.com/mozilla-mobile/firefox-android
https://github.com/mozilla-mobile/firefox-android
Mozilla Public License 2.0
6.48k stars 1.27k forks source link

For #1813787 - Hide `Add to shortcuts` menu entry if Shortcuts are disabled on the Homepage #28757

Closed gitstart closed 1 year ago

gitstart commented 1 year ago

"Add to shortcuts" menu entry should be hidden if Shortcuts are disabled on the HomepageScreens need refactor.

Issue Screenshots

fnx-54-issue.webm

Fix Screenshots

fnx-54-fixed.webm

Pull Request checklist

QA

GitHub Automation Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1813787 Fixes #1813787

firefoxci-taskcluster[bot] commented 1 year ago
No Taskcluster jobs started for this pull request ```js The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the default branch) does not allow starting tasks for this pull request. ```
csadilek commented 1 year ago

Hey @gitstart, Thank you so much for your contribution. It’s much appreciated. Unfortunately, we are in the process of moving this repository to firefox-android early next week, and won’t be able to review or land any PRs until after the migration. Please feel free to reopen a new PR against the new repo once migration is complete. You will have to clone the new repository and reapply your changes. Thank you again, and sorry for the inconvenience.

For more information, please see https://github.com/mozilla-mobile/fenix/issues/26855.