Skip to content

Dedicated link creation dialog#41

Closed
mberry wants to merge 13 commits into
feature/vitejs-sveltefrom
dedicated-link-creation-dialog
Closed

Dedicated link creation dialog#41
mberry wants to merge 13 commits into
feature/vitejs-sveltefrom
dedicated-link-creation-dialog

Conversation

@mberry

@mberry mberry commented Sep 18, 2023

Copy link
Copy Markdown

Added save button which opens link creation form
Will also open on ctrl+v and paste the url
Only shows url field at first, then autofills with title
Has a 1500ms debounce to keep it from opening prematurely if the user
is typing the address.
Closes dialog form after link created

gwendalF and others added 12 commits September 3, 2023 17:00
Also removed the static  ARK_SHELF_DATA_PATH
Added the yarn clean script to clean the cargo cache
Will also open on ctrl+v and paste the url
Only shows url field at first, then autofills with title
Has a 1500ms debounce to keep it from opening prematurely if the user
is typing the address.
Closes dialog form after link created
@mberry mberry changed the base branch from main to feature/vitejs-svelte September 18, 2023 03:54
@gwendalF gwendalF force-pushed the feature/vitejs-svelte branch from be6c40d to def6b81 Compare September 18, 2023 16:03
@kirillt

kirillt commented Sep 18, 2023

Copy link
Copy Markdown
Member

Is it correct to imagine that the "AUTO FILLED" button is invisible now and it automatically pressed every 1.5s?

@mberry

mberry commented Sep 19, 2023

Copy link
Copy Markdown
Author

It sets a timer the moment a key is pressed, if the key is pressed again it resets. If the timer hits zero it makes the network connection to autofill.

@kirillt

kirillt commented Sep 23, 2023

Copy link
Copy Markdown
Member

@mberry we need update this PR to catch up with main. By the way, let's target main in all PRs for the time being.

@mberry

mberry commented Oct 1, 2023

Copy link
Copy Markdown
Author

Replaced by #47

@mberry mberry closed this Oct 1, 2023
@kirillt kirillt deleted the dedicated-link-creation-dialog branch February 6, 2024 10:58
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.

4 participants