chore(blocks): refine linked web toolbar

toeverything / blocksuite

🧩 Content editing tech stack for the web - BlockSuite is a toolkit for building editors and collaborative applications.

https://blocksuite.io

Mozilla Public License 2.0

4.11k stars 358 forks source link

Closed fundon closed 1 week ago

fundon commented 2 weeks ago

Closes: BS-583

vercel[bot] commented 2 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
blocksuite	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 10, 2024 7:59am

1 Skipped Deployment

graphite-app[bot] commented 2 weeks ago

Add the label “merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

fundon commented 2 weeks ago

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fundon and the rest of your teammates on Graphite

Svaney-ssman commented 1 week ago

UI Issues:

Need to display the url in toolbar more concisely, please refer to the link display in card view: check in figma
the default button of " inline/card/embed transition" has no background, this is a hover state

fundon commented 1 week ago

Need to display the url in toolbar more concisely, please refer to the link display in card view: check in figma

So only domain names are displayed here?

Svaney-ssman commented 1 week ago

Visit Preview

Yes! please align text to the left CleanShot 2024-07-05 at 10 35 47@2x CleanShot 2024-07-05 at 10 37 30@2x

Svaney-ssman commented 1 week ago

It seems the font weight and button default state have not changed yet CleanShot 2024-07-05 at 10 42 38@2x

graphite-app[bot] commented 1 week ago

Jul 10, 3:49 AM EDT: Saul-Mirone added this pull request to the Graphite merge queue.
Jul 10, 4:07 AM EDT: Saul-Mirone merged this pull request with the Graphite merge queue.