This repository has been archived by the owner on Dec 11, 2019. It is now read-only.
ledgerTable with pinned sites does not have the orange border #10434
Assignees
Labels
Milestone
Comments
8 tasks
dfperry5
pushed a commit
to dfperry5/browser-laptop
that referenced
this issue
Aug 18, 2017
Fixes brave#10434 Follow-up to brave#10265 Auditors: @cezaraugusto Test Plan: 1. Run `npm run add-simulated-synopsis-visits 100` 2. Open about:preferences#payments 3. Pin some sites 4. Make sure the orange border appears on the table
This was referenced Dec 20, 2017
This was referenced Dec 27, 2017
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Test plan
#10435 (comment)
Describe the issue you encountered: ledgerTable with pinned sites does not have the orange border.
This is because I forgot to include borderCollapse: 'collapse' on ledgerTable.js to sortableTable.js on #10265.
Platform (Win7, 8, 10? macOS? Linux distro?): All
Brave Version (revision SHA): master branch
Steps to reproduce:
npm run add-simulated-synopsis-visits 100Actual result: the orange border does not appear on the table
Extra QA steps:
Any related issues:
The text was updated successfully, but these errors were encountered: