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

Delay file creation when a new file is created in the file manager. #2339

Merged

Conversation

drgrice1
Copy link
Member

Instead of creating the file when the new file confirmation page is submitted, create the file when "Save" (or "Save As") is clicked from the edit page that opens next.

This was requested in issue #2326.

Also, don't focus the name input on the edit page. That is a bit annoying. Only focus that input on the confirmation page. That is the only page that has a data input.

Copy link
Member

@pstaabp pstaabp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@drgrice1 drgrice1 force-pushed the file-manager-new-file-delay-create branch from dd8f462 to aa71868 Compare February 29, 2024 13:50
Instead of creating the file when the new file confirmation page is
submitted, create the file when "Save" (or "Save As") is clicked from
the edit page that opens next.

This was requested in issue openwebwork#2326.

Also, don't focus the `name` input on the edit page.  That is a bit
annoying.  Only focus that input on the confirmation page.  That is the
only page that has a `data` input.
@drgrice1 drgrice1 force-pushed the file-manager-new-file-delay-create branch from aa71868 to 9e37b12 Compare February 29, 2024 21:26
@Alex-Jordan
Copy link
Contributor

Works as described, thanks!

@Alex-Jordan Alex-Jordan merged commit b150659 into openwebwork:develop Feb 29, 2024
2 checks passed
@drgrice1 drgrice1 deleted the file-manager-new-file-delay-create branch February 29, 2024 22:20
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

Successfully merging this pull request may close these issues.

3 participants