-
Notifications
You must be signed in to change notification settings - Fork 97
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
Archive forms #1834
Comments
Hi @j0hannesr0th and thanks for your offer to contribute! For me, this sounds like a good idea. While you're at it, you could perhaps also implement #1101: Disabling would keep the form in the normal navigation, but no longer allow submissions. Archiving would hide the form from the normal navigation and also no longer allow submissions. I'm unsure if we should use two flags @susnux @jotoeri What do you think about it? @nextcloud/designers Any suggestions from a design perspective on how it should look like? |
@Chartman123 for me it's ok to not add a second state, so I'll use |
@j0hannesr0th I've talked to @susnux in the meantime. We both think that it makes sense to implement this the following way:
Regarding the UI, I'll ping the designers once more :) I'm unsure if we should add the UI elements to the settings sidebar or to the navigation menu or to both... |
In the navigation we have "Your forms" and "Shared with you", we could add another section "Archived forms" below this? Btw, do we use AppNavigationCaption there? Cause e.g. unlike in Contacts, the headings are not blue (color-primary-element) |
I like the way it is implemented in the link in the OP here :)
Yes, we're using AppNavigationCaption without any additional styling... |
Seems like contacts is using old version of |
@j0hannesr0th an issue with the implementation example is that it creates a mode toggle. It is not directly obvious whether one is in the archive or in the active projects. What would be preferable is if "Archived forms" opens a modal with a list much like "Deleted calendars" does. Additionally, there is no need to have a counter on the archive since counters are calling for attention, e.g. if something is new or unread. Does that work @j0hannesr0th @susnux @Chartman123? :) |
@jancborchardt In the latest released version, I've updated the icons - see video below. This makes it clear whether a project is archived or not. I'm not in favor of the modal concept. The toggle option is also much quicker. @Chartman123 After considering your suggestion, I've decided it's better to utilize the isDisabled state. I manage numerous forms and sought a simple filter to display only old/disabled projects. For archival and revision purposes, I prefer not to delete these forms, but also wish to keep them hidden. Here's my approach:
Idea: implement some kind of filter input, which filters visible projects by name while typing (can be done in second step also): Released version of the cospend implementation: 2023-12-25_171115.mp4 |
@j0hannesr0th Please create a PR in this repository for your WIP, so that it's easier for us to see what you're doing. Also please don't mix any feature additions in a single PR. This one should be about archiving forms only and not about filtering... And as pointed out above we'd like to |
@Chartman123, there isn't a work-in-progress (WIP) version yet, as I first need to determine what should be implemented and how. By archiving forms, I mean that I want to view only "active" projects. I have many forms that are currently not in use, but I wish to retain them for revision purposes. Introducing a new state isArchived seems to add unnecessary complexity, in my opinion. With this, toggling becomes impractical due to the possible combinations: isDisabled = True, isArchived = True So, to clarify: I don't intend to add an archive feature. Instead, I simply want to switch between active and inactive projects. |
Yes, I understand what you mean, but we want to have this the way we pointed out in the discussion above. Not everyone wants to have all "inactive" forms hidden. So we will need a distinction. Therefore the So there are basically only three possible combinations:
|
Hi @Chartman123 I've forked this and implemented it the way I wanted/needed it - without extra state and with toggle function. I can merge that. If you don't want that, I'll unassign this issue. |
@j0hannesr0th Ok, then thanks, but no, we want to have it the way we elaborated here in the discussion. So no backport needed from your fork. |
Nextcloud (please complete the following information):
Is your feature request related to a problem? Please describe.
If you have many forms that may be outdated but you want to keep them anyway, an archive function would solve that problem.
Describe the solution you'd like
I have implemented an archive function in the "Cospend" project; you can find more details and a video at julien-nc/cospend-nc#236. I plan to introduce this same functionality in this project.
If this is acceptable to you, please let me know, and I will begin development.
Describe alternatives you've considered
none.
Additional context
none.
The text was updated successfully, but these errors were encountered: