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

Pinning integration: Stub UI for Settings screen #1501

Closed
2 of 4 tasks
jessicaschilling opened this issue May 28, 2020 · 5 comments · Fixed by #1685
Closed
2 of 4 tasks

Pinning integration: Stub UI for Settings screen #1501

jessicaschilling opened this issue May 28, 2020 · 5 comments · Fixed by #1685
Assignees
Labels
area/pinning Integrating pinning into GUI apps effort/days Estimated to take multiple days, but less than a week exp/intermediate Prior experience is likely helpful kind/enhancement A net-new feature or improvement to an existing feature P0 Critical: Tackled by core team ASAP status/in-progress In progress topic/design-content Content design, writing, information architecture topic/design-front-end Front-end implementation of UX/UI work topic/design-ux UX strategy, research, not solely visual design topic/design-visual Visual design ONLY, not part of a larger UX effort

Comments

@jessicaschilling
Copy link
Contributor

jessicaschilling commented May 28, 2020

Note: This issue is part of a larger pinning service integration epic undertaken spring/summer 2020.

In brief

Stub out UI for Settings screen of Web UI/Desktop, as follows:

  • Create high-fidelity Figma doc displaying all steps in process, including errors/modals/etc
  • Build in the display (but not functionality) of new Settings screen elements as depicted in Figma
  • Add the minimum amount of interactivity required to be able to travel through the Files screen to demonstrate said elements
  • Create required documentation (to be homed on docs.ipfs.io) as linked to from within the UI

High-res screens as Figma doc are here.

@jessicaschilling jessicaschilling added the area/pinning Integrating pinning into GUI apps label May 28, 2020
@jessicaschilling jessicaschilling added kind/enhancement A net-new feature or improvement to an existing feature P0 Critical: Tackled by core team ASAP exp/intermediate Prior experience is likely helpful effort/days Estimated to take multiple days, but less than a week status/in-progress In progress topic/design-content Content design, writing, information architecture topic/design-front-end Front-end implementation of UX/UI work topic/design-ux UX strategy, research, not solely visual design topic/design-visual Visual design ONLY, not part of a larger UX effort labels May 28, 2020
@jessicaschilling
Copy link
Contributor Author

@rafaelramalho19 @lidel Per discussion earlier today -- Updated Figma high-res screens to include the full settings flow:

  • Screen: First load/no third-party services
  • Modal: select service provider
  • Modal: configure known service provider
  • Modal: configure custom service provider
  • Screen: local pinning plus several third-party service providers
  • Pop-up: dots menu for a service provider
  • Modal: remove a service provider
  • Screen: confirm service provider was removed

Would appreciate feedback/discussion with you both before we take this wider. Thanks!

@jessicaschilling
Copy link
Contributor Author

@rafaelramalho19 @lidel -- Thanks for the feedback, all is now incorporated in the Figma high-res screens. I'll move on to the Files screens under #1500, since these should tentatively be ready to stub out.

cc @momack2

@jessicaschilling
Copy link
Contributor Author

ETA: Considerable revisions to the document based on in-Figma comments. Getting much closer but we still need to determine whether we'll be allowing manual pinning in this release.

Awaiting consensus from @rafaelramalho19 @lidel @momack2.

@jessicaschilling
Copy link
Contributor Author

Per meeting with @lidel and @rafaelramalho19 -- we should be good to start stubbing this out.

@jessicaschilling
Copy link
Contributor Author

@rafaelramalho19 -- Can you please make the following adjustments to the Settings screen based on our meeting this morning? I'll drop screenshots in here, but the Figma source docs have been updated as well.

  1. In the table, please remove the columns for bandwidth and storage used, and replace them with a "Pins" column that will just display the number of pinned files on each service.
    image

  2. Can you please reduce the number of options for "auto upload" rules to two: "Disabled", and "All files"? This manifests in two locations: the "Auto Upload" column of the grid (see above screenshot) and in the config modal (see below screenshot), where the options in the dropdown would be "All files on local node" or "Disabled".
    image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/pinning Integrating pinning into GUI apps effort/days Estimated to take multiple days, but less than a week exp/intermediate Prior experience is likely helpful kind/enhancement A net-new feature or improvement to an existing feature P0 Critical: Tackled by core team ASAP status/in-progress In progress topic/design-content Content design, writing, information architecture topic/design-front-end Front-end implementation of UX/UI work topic/design-ux UX strategy, research, not solely visual design topic/design-visual Visual design ONLY, not part of a larger UX effort
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants