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

Change "Copy" AND "Paste" to just "Duplicate" #1984

Closed
pikurasa opened this issue Dec 24, 2019 · 12 comments · Fixed by #2061
Closed

Change "Copy" AND "Paste" to just "Duplicate" #1984

pikurasa opened this issue Dec 24, 2019 · 12 comments · Fixed by #2061

Comments

@pikurasa
Copy link
Collaborator

I have observed that in the class, the extra step to click "paste" has its own learning curve (albeit small).

When I think of copy and paste, I think the main advantage to separating the two steps is that the user has some control over what they copy (i.e. the program remembers what is on clipboard or the user can copy multiple--sometimes non-contiguous--items). We do not really have these features.

In the absence of such features, do we really want to keep the two steps separate? Or do we want one "duplicate" feature? (Or perhaps we call it "copy" as it seems that students often expected this single step to be sufficient in duplicating the given block/s)

@walterbender
Copy link
Member

I think this make a lot of sense. If they want multiple copies, they can hit duplicate repeatedly.

@pikurasa pikurasa changed the title Do we need Copy AND Paste, or do we just need "Duplicate" Change "Copy" AND "Paste" to just "Duplicate" Dec 26, 2019
suyashsingh234 added a commit to suyashsingh234/musicblocks that referenced this issue Dec 29, 2019
suyashsingh234 added a commit to suyashsingh234/musicblocks that referenced this issue Dec 29, 2019
@pikurasa
Copy link
Collaborator Author

My last request is that the pie menu stay open so that the user can duplicate multiple times without having to right click every time.

Screenshot at 2020-01-16 09:47:07 copy should stay open

Thank you!

@pikurasa pikurasa reopened this Jan 16, 2020
@arvindh75
Copy link

Can I start working on this ?

@aviral243
Copy link
Member

I need to change this property right?

Screenshot from 2020-01-22 01-52-21

@walterbender
Copy link
Member

Just comment out that line and it should work.

@aviral243
Copy link
Member

@walterbender After doing the changes, how will they reflect on the development server. I commented out the line, reloaded the page and nothing changed.
Please help

@walterbender
Copy link
Member

Are you testing from your local file system?

@aviral243
Copy link
Member

I don't have an idea on this. I'm new to the code. Can you please guide me a little bit on this?

@walterbender
Copy link
Member

Perhaps you can do the github tutorial on how to clone a repository?

Then make a branch, make your changes, and test them in your browser:

file:///home/aviral/musicblocks/index.html

@aviral243
Copy link
Member

Thanks. Figured it out. I was using npm run serve and visiting localhost:3000 to see changes.
Can I make a pull request ?

@walterbender
Copy link
Member

You are welcome to make a PR. Please be sure to follow the guidelines here:
https://github.com/sugarlabs/sugar-docs

@aviral243
Copy link
Member

Sure.

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.

4 participants