mirror of
https://github.com/go-vikunja/vikunja.git
synced 2026-03-09 07:13:35 -05:00
Improvements for Link Sharing Modal #423
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/vikunja#423
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @vikunja-bot on GitHub (Apr 1, 2025).
Original issue by dpschen on 2022-09-14T15:55:23.000Z
Copying over from https://kolaente.dev/vikunja/frontend/pulls/2335#issuecomment-35155
When testing the sharing functionality again I stumbled over a few issues:
Unsure: move the 'Create a new link share'-Button and the create-form below the list of already created shares. This way it's clear that the newly created share is added to the bottom of the list. Possible downside: out of view on mobile and small desktop windows.
Non reversible UI action: Clicking the 'Create a new link share'-Button should be reversible if you clicked there by mistake. Aka: add a 'Cancel' or 'x' button in the inline form.
Inline form the create a new share link is hard to tell apart from the rest of the form.
Missing space between 'Cancel' and 'Create' button. This might be a sideeffect from some of my Modal changes, but might make sense to look into this here. Seems like a
gapproperty is missing for.card-footer.When clicking the copy button it should be indicated that the new link is now in the clipboard. E.g. marking the copy button green and adding checkmark or success message. Would prefer the checkmark since this way the user doesn't loose context and the feedback is immediately and inline. After a few seconds the button should go back in its initial state. This button or maybe the whole input should be abstracted dedicated component
This is a general issue on mobile: It's hard to make the tooltip of 'What is a share link?' appear. Tapping makes it appear but it will directly disappear again. Since the info is so short I think we can remove the tooltip here completely and simply show the explaining text.
Picky: The tap area of the copy button seems to small on mobile. It would benefit from an
aspect-ratio: 1/1;Original issue on Gitea
xeruf commented on 2022-09-24T23:15:45.000Z:
I guess I'll just tack this on here: Visiting something like
/lists/43/settings/sharecreates an uncloseable popup ^^@kolaente commented on 2022-09-26T12:23:58.000Z:
@xeruf Is that reproducible on try?
xeruf commented on 2022-09-27T15:04:49.000Z:
well, not uncloseable there, but still unexpected: https://try.vikunja.io/list/1/settings/share
because it just says not found
however, when it points to a list not created by myself, I get this uncloseable popup
@kolaente commented on 2022-09-27T15:06:07.000Z:
That url should be
/lists/1/settings/share(notice the additionalsforlists).xeruf commented on 2022-09-28T10:12:51.000Z:
Ah right, I keep making that mistake. Nonetheless, try opening these in a new tab an clicking the x in the popup:
https://try.vikunja.io/lists/1/settings/edit
https://try.vikunja.io/lists/1/settings/share
It tries to navigate back, which is unexpected.
I think vikunja should check whether the previous page is on the same domain, otherwise just show the list on close.
dpschen commented on 2022-09-28T10:30:54.000Z:
I agree with this. I see this problem everywhere where we use
router.back().I'm not sure if checking the domain is the right approach. Maybe we should check if the last history entry contains the app base path instead
xeruf commented on 2022-09-28T10:43:47.000Z:
Exactly that is what I meant :)
Furthermore, my colleague just had an issue with his jwt token being invalidated when he clicked on the link, logged in, and closed the popup - it looked like he was logged in, but all lists appeared empty. Very confusing.
@kolaente commented on 2022-09-28T13:38:06.000Z:
That sounds like a good approach. I've added an item in the backlog for this.
dpschen commented on 2022-11-22T11:49:57.000Z:
I try to tackle the modal navigation issues with https://kolaente.dev/vikunja/frontend/pulls/2735