Skip to content
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

Improvements for the IPFS diagnostics page brave://ipfs #13173

Closed
9 tasks done
bbondy opened this issue Dec 11, 2020 · 3 comments · Fixed by brave/brave-core#7948
Closed
9 tasks done

Improvements for the IPFS diagnostics page brave://ipfs #13173

bbondy opened this issue Dec 11, 2020 · 3 comments · Fixed by brave/brave-core#7948

Comments

@bbondy
Copy link
Member

bbondy commented Dec 11, 2020

This issue was broken out from @jessicaschilling's issue posted here: #13125

brave://ipfs diagnostic screen changes:

2020-12-05--19-28-17

  • Start and stop button should only show when it is applicable (Improve start and stop button behaviour for brave://ipfs #14115)

  • Add Restart button (speeds up applying config changes)

  • If not installed, note "IPFS not installed" and button to install and start; other items should be grayed out to indicate missing functionality

  • Additionally, display the Learn more about IPFS and privacy link described above (placement/appearance TBD once we know whether this diagnostic page can be included with the rest of Brave settings UI)

  • Diagnostic page displays API address, but there is no way to open Web UI

    • Fix: when node is running, add a button that opens its Web UI (http://127.0.0.1:{api-port}/webui)
      • Display at same level of prominence as Launch/Shut Down button
      • Q: Text should be My Node to match IPFS Companion
      • Q: Button should be a different color to convey different function than Launch/Shut Down button — will be able to have better guidance on this once we know whether this diagnostic page can be included with the rest of Brave settings UI
  • Connected peers: Add (details) link after peer count, linking to Peers screen in IPFS Web UI (http://127.0.0.1:{api-port}/webui/#/peers)

  • If the user has this diagnostic page open in one tab with their node stopped, and then starts their node from a different tab, this tab doesn't update unless it's reloaded. Can this be fixed?

  • When diagnostic page is opened on an empty profile (i.e. user has not yet been asked to enable IPFS support), there is no UI to start the node:

    2020-12-05--19-24-19

    • Fix (this behavior is already specified earlier in this document): If not installed, note “IPFS not installed” and button to install and start; other items should be grayed out but present to indicate missing functionality
    • Additionally, display the Learn more about IPFS and privacy link described above (placement/appearance TBD once we know whether this diagnostic page can be included with the rest of Brave settings UI)
@bbondy bbondy added priority/P4 Planned work. We expect to get to it "soon". OS/Desktop labels Dec 11, 2020
@jessicaschilling
Copy link

Some of the notes above hinged on the question about whether it was possible to include this content in the main Brave settings pages instead. Since that sounds like it's not the best way forward for now, suggesting some amendments to the checklist above for clarity:

  • Replace Daemon status with IPFS node status
  • Replace Launched: true with Node is running or Node is not running
  • Buttons should read Start or Stop instead of Launch or Shutdown (think this was actually covered under IPFS text updates #13169)
  • Learn more about IPFS and privacy link (to https://support.brave.com/hc/en-us/sections/360010974932-InterPlanetary-File-System-IPFS-) should appear on its own line under the list of addresses, with some padding-top to separate it
  • Button for My node should be a different color from the Start or Stop button - is there an existing visual style for this?

@jessicaschilling
Copy link

Note: please see #13170 (comment) for a quick mockup that illustrates some of the above requests. Thanks!

@stephendonner
Copy link

Verified FIXED using beta and the inline test notes here, on

Brave 1.22.51 Chromium: 89.0.4389.72 (Official Build) beta (x86_64)
Revision 3f345f156bfd157bd1bea06310e55f3fb2490359-refs/branch-heads/4389@{#1393}
OS macOS Version 11.2.2 (Build 20D80)

Start, Stop, and Restart buttons were verified in #14115

  • default state, with 0'd out stats and empty paths:

Screen Shot 2021-03-03 at 9 15 17 AM

  • after clicking Install and start:

Screen Shot 2021-03-03 at 9 17 11 AM

Screen Shot 2021-03-03 at 9 17 48 AM

@stephendonner stephendonner added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants