-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
refactor(web): descriptions #6517
Conversation
Deploying with Cloudflare Pages
|
{#if !isOwner || api.isSharedLink} | ||
<span class="break-words">{description}</span> | ||
{:else} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you get rid of this span and convert it back to a placeholder? Did you test a multi-line description in a shared link, specifically going from short to long to see if it auto grows?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't work and re-introduces back an auto grow bug. In this video I click "next" to go to another asset with a longer description. The textarea does not auto-grow until I start typing.
Screencast.from.01-19-2024.09.08.42.PM.webm
Oh right, I see what's the issue. Should be fixed. Thank your for the explanation Screencast.from.2024-01-20.03-19-55.webm |
b3895ba
to
32073fa
Compare
32073fa
to
00f3e56
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello, thank you for your work on this. It works very well.
One bug I notice is that you cannot see the description of the non-owned asset in a shared album.
@martabal Here is what I see. Probably a bug with a very long description |
ff28ba7
to
adfc81d
Compare
Really weird, I can't reproduce it. I tested all kind of descriptions (none, long, one line) with assets in shared albums, viewed from the owner, a non-owner and a guest and I can't reproduce this behavior 🤔 I changed a bit that logic in my latest commit, can you try it ? |
@martabal the description is still showing up as a single line if you share the album in public shared link |
Right, I forgot to update the description in the Thanks for the info |
What's changed
Ctrl + Enter
Screenshots
2024-01-20.01-07-24-2.mp4