-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Website added via Add site modal is not added to NTP #24370
Labels
bug
feature/new-tab
OS/Desktop
polish
Nice to have — usually related to front-end/visual tasks
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-macOS
QA Pass-Win64
QA/Test-Plan-Specified
QA/Yes
release-notes/exclude
Milestone
Comments
MadhaviSeelam
changed the title
website added via Add site modal is not added to NTP
Website added via Add site modal is not added to NTP
Aug 1, 2022
rebron
added
the
priority/P3
The next thing for us to work on. It'll ride the trains.
label
Aug 1, 2022
Sounds quite similar to brave/brave-core#14293. Would you mind checking in the latest Nightly to see if it reproduces? |
@fallaciousreasoning yes, it is reproducible in 1.44.12.
|
👍 I'll take a look |
fallaciousreasoning
added
the
polish
Nice to have — usually related to front-end/visual tasks
label
Aug 1, 2022
25 tasks
stephendonner
added
QA/Test-Plan-Specified
bug
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
labels
Sep 2, 2022
Verified
|
Brave | 1.44.73 Chromium: 105.0.5195.68 (Official Build) beta (x86_64) |
---|---|
Revision | ad13e82529051bac6a0e65f455e6d7a1e5fd7938-refs/branch-heads/5195@{#903} |
OS | macOS Version 13.0 (Build 22A5331f) |
Steps:
- installed
1.44.73
- launched Brave
- loaded
bbc.com
- open a new tab page
- confirmed
bbc.com
is added to the NTP - clicked
...
in the same view box/row as the websites - clicked
Add site
- added website information in the modal (
usatoday.com
) and clickedSave
- clicked on the
usatoday.com
favorite
Confirmed it made and successfully loaded the usatoday.com
favorite
example | example | example | example | example | example |
---|---|---|---|---|---|
stephendonner
added
QA Pass-macOS
and removed
QA/In-Progress
Indicates that QA is currently in progress for that particular issue
labels
Sep 2, 2022
Verification
Followed my original steps from the description
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
feature/new-tab
OS/Desktop
polish
Nice to have — usually related to front-end/visual tasks
priority/P3
The next thing for us to work on. It'll ride the trains.
QA Pass-macOS
QA Pass-Win64
QA/Test-Plan-Specified
QA/Yes
release-notes/exclude
Description
Steps to Reproduce
...
in the same view box/row as the websitesAdd site
Actual result:
Website (
usatoday
) is not shown on NTP2022-08-01_14h27_58.mp4
Expected result:
Website should be shown along with correct favicon
Reproduces how often:
Easily
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
@fallaciousreasoning @rebron
The text was updated successfully, but these errors were encountered: