feat: harden textarea auto height algorithm #985
No reviewers
Labels
No Label
area/internal-code
changes requested
confirmed
dependencies
duplicate
good first issue
help wanted
hosting
invalid
kind/bug
kind/feature
question
wontfix
No Milestone
No project
No Assignees
3 Participants
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vikunja/frontend#985
Loading…
Reference in New Issue
No description provided.
Delete Branch "dpschen/frontend:feature/harden-textarea-auto-height"
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?
tested on current chrome, firefox, safari
Hi dpschen!
Thank you for creating a PR!
I've deployed the changes of this PR on a preview environment under this URL: https://985-featureharden-textarea-auto-height--vikunja-frontend-preview.netlify.app
You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somehwere. The easiest to use is https://try.vikunja.io/.
Have a nice day!
feat: harden textarea auto height algorythmto feat: harden textarea auto height algorithmIt looks like this breaks on Firefox mobile (android).
Whoops, didn't mean to approve
Seems like 'harden' was the wrong term :P
Will check
7b336d1fda
tob8df6f2fb7
@ -7,3 +7,3 @@
class="input"
:placeholder="$t('list.list.addPlaceholder')"
cols="1"
rows="1"
Maybe this line fixes the issues in android.
cols="1"
was wrong either way :DStill broken :/ this time on firefox desktop (v89.0) as well.
:/
f150bd732f
toa6234a5806
Also it looks like I completely messed up the git log of this branch when I merged main into it. Should be fine now with a rebase...
a6234a5806
tob8df6f2fb7
Still broken in Firefox.
Yes, sry I'm still on this. I just wanted a deployment so that I can test it better =)
Ah makes sense. Looks like I was too enthusiastic with wanting to re-review it.
Just ping me if you're ready :)
b8df6f2fb7
to8850ac9bf5
You can retry, but I didn't test. How can I force a redeploy from @frederick ?
Seems to work now!
There just one small issue I have with it: After adding a bunch of tasks the input stays the same height:
It should do one automatically. Looks like it didn't, I've restarted the build pipeline. Maybe that will do.
This pull request is starting to get in my top 10 list of most complex pull requests for most meaningless feature :D
(To be clear that's more self critic than critic in the review style. I really like the latter =) )
8850ac9bf5
to082aba042b
I hope this fixes it.
Interesting: I didn't know but there is no way except manual pulling (via setTimout) to check for changes of a textareas value.
The issue that you showed above come from setting the value manually (via vue's bindings).
While I was it I ported the component to typescript and script setup.
082aba042b
toed9a29dfc9
Really nice!
It's beautiful 😁