-
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
Can't restore sites in ac table #1883
Labels
feature/rewards
priority/P5
Not scheduled. Don't anticipate work on this any time soon.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Yes
release-notes/include
Milestone
Comments
@jenn-rhim do you have any direction on where the restore action could go inside the Box panel? This should be an easy add: |
NejcZdovc
added
the
priority/P5
Not scheduled. Don't anticipate work on this any time soon.
label
Oct 31, 2018
@ryanml posting the mockup reflecting what we discussed yesterday... thanks @NejcZdovc. very close. the only difference is to maintain the messaging clarity as in the overlay. |
18 tasks
Verified passed with
Verification passed on
Test plan from brave/brave-core#835 Verification passed on
|
kjozwiak
added
release-notes/exclude
release-notes/include
and removed
release-notes/exclude
labels
Dec 18, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
feature/rewards
priority/P5
Not scheduled. Don't anticipate work on this any time soon.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Yes
release-notes/include
Description
Steps to Reproduce
Actual result:
You don't see restore option until you add at least 5 sites.
Expected result:
You should see restore before 5 sites as well and hide that option after 5 sites
Reproduces how often:
Brave version (brave://version info)
Reproducible on current release:
Website problems only:
Additional Information
The text was updated successfully, but these errors were encountered: