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

M1: Basic Component Copy-Paste Proof-of-Concept - Studio Prototype #1

Open
15 tasks done
ormsbee opened this issue Nov 15, 2022 · 7 comments
Open
15 tasks done
Assignees
Labels
epic Large unit of work, consisting of multiple tasks

Comments

@ormsbee
Copy link

ormsbee commented Nov 15, 2022

A Studio prototype for component copy/paste will be built on edx.org.

Should handle pasting both within the same course as well as across courses, but not using libraries yet.

M1 gets the very basic functionality out there in a way that’s testable by users, but is not really releasable to a production environment.

@jmakowski1123 jmakowski1123 added the epic Large unit of work, consisting of multiple tasks label Nov 15, 2022
@jmakowski1123 jmakowski1123 changed the title M1: Basic Component Copy-Paste Proof-of-Concept M1: Basic Component Copy-Paste Proof-of-Concept - Studio Prototype Feb 7, 2023
@ali-hugo
Copy link

@bradenmacdonald Here is a link to the design files in Figma. I will add the svg's of all the icons used to this ticket as well.

@jmakowski1123 I have made the last few UX changes that we discussed in our meeting last week. Would you mind taking one last look over the designs and confirming whether you'd like any changes before Braden implements the design? I think the only thing outstanding is the wording of the error/warning/info alerts.

@ali-hugo
Copy link

ali-hugo commented Feb 13, 2023

SVG Icons

  • More Vert: Image
  • Content Paste: Image
  • Help: Image
  • Error: Image
  • Info: Image
  • Warning: Image

@ali-hugo
Copy link

UX of alerts

The UX of the alerts is design to work like the edX registration form at this link. Notice how the screen scrolls up to display the alert at the top of the screen.

@jmakowski1123
Copy link

jmakowski1123 commented Feb 15, 2023

Thanks @ali-hugo ! Is this the final outstanding question you are referring to? If so, flagging @ormsbee on that one.

From my perspective, these look good to go! @bradenmacdonald @ormsbee I've updated the Technical Approach document, specifically the "Summary of New UI elements" section, so that it reflects all of the new required UI for the Studio prototype. Would appreciate an extra set of eyes on that, to make sure I didn't miss anything. Tracking that specific task here.

@bradenmacdonald
Copy link
Contributor

bradenmacdonald commented Feb 15, 2023

Perfect, thanks for all these links and for the SVGs, @ali-hugo !

@ali-hugo would it be possible for you to show how the "Paste Component" button should look when nothing has been copied or what's in the clipboard is not suitable to paste as a component (e.g. in the future when a unit is copied, it can't be pasted as a component). Basically this question: https://openedx.atlassian.net/wiki/spaces/OEPM/pages/3575611457?focusedCommentId=3584851970

I also had one question at https://www.figma.com/file/y2qfzidTxxfYK8ONgyT7KP?node-id=698:1155#370741457

@ali-hugo
Copy link

ali-hugo commented Feb 16, 2023

@jmakowski1123 Thanks for flagging Dave on that comment. I see he's already responded. Here is a list of the last few unresolved conversations:

  • Would components ever show a loading state when pasting? (see comment)

  • Is having some kind of "troubleshooting guide" pragmatic for MVP, or should this alert text rather just encourage the user to contact their system administrator? (see comment)

  • Could the display of alerts / inline notifications be simplified for MVP? Braden's question (see comment)

@jmakowski1123 @bradenmacdonald @ormsbee Feel free to respond to these comments here or in Figma - whichever is easiest for you.

@ali-hugo
Copy link

ali-hugo commented Feb 16, 2023

@bradenmacdonald

@ali-hugo would it be possible for you to show how the "Paste Component" button should look when nothing has been copied or what's in the clipboard is not suitable to paste as a component

We decided to simply not display the paste button if a paste is not possible. There was quite a bit of back-and-forth about this but, in the end, we decided to try it out in user-testing and see how users react.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
epic Large unit of work, consisting of multiple tasks
Projects
Status: Done
Development

No branches or pull requests

4 participants