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

[Feature Request] Give user choice to view forked copy or fork again #2685

Open
imranariffin opened this issue Feb 12, 2020 · 4 comments
Open

Comments

@imranariffin
Copy link
Collaborator

imranariffin commented Feb 12, 2020

As a user I would like to know that a notebook I'm interested to copy/fork has already already been copied/forked by me.

What I Did

Viewed a notebook, forked and edited it only to later realize that I had already forked it before.

What I Expected

If I had done some modifications on the first copy in some cases I would rather continue my work there than starting over. I would like to have a choice to either continue work on the existing fork or fork a new one.

As per discussion in PR #2550, it might be a good idea to roughly follow GitHub's forking behavior:
github-fork-modal

In our case we would probably either offer to open up existing fork(s) in a new tab or allow the user to fork again. You could probably do this with a modal element. A bit complicated but might be a fun project!

So this is the behavior that I propose:

When a user who has already forked this notebook clicks on the "Make a copy to your account" link, open a modal instead of immediately forking the notebook.
The modal will display:

  1. A list of forked copies of the notebook that user can click to open the notebook in a new tab
  2. A button to allow the user fork another copy of the notebook
  3. A button at top right to close the modal

If the user has not forked the notebook previously, then the current behavior should be used (immediately copy the notebook).

What Happened

Now I have two copies of the same notebook with different changes.

Screenshot from 2019-12-30 21-35-50

(if applicable) notebook that reproduces bug

Applicable to any notebook that you have already forked.

@bcolloran
Copy link
Contributor

thanks for filing this issue @imranariffin !

@imranariffin
Copy link
Collaborator Author

@bcolloran @wlach
Are we going forward with this requested feature? Just needed to double check before I start working on it.

@bcolloran
Copy link
Contributor

Yea, I think it sounds like a good idea @imranariffin

I'd suggest clarification of the behavior: the modal should only open if the user has previously forked the NB being viewed; If the user has not forked the notebook previously, then the current behavior should be used (immediately copy the notebook)

@imranariffin
Copy link
Collaborator Author

imranariffin commented Feb 27, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants