tablelandnetwork / studio

Discover, design, deploy, and manage data driven web3 apps on Tableland.
https://studio.tableland.xyz
4 stars 1 forks source link

(web) show project id for cli users #228

Closed joewagner closed 6 months ago

joewagner commented 6 months ago

fixes https://linear.app/tableland/issue/ENG-831/studio-web-app-should-show-me-project-id-somewhere

If you are trying to use the CLI in combination with the web app, it will be helpful to have a way to find and copy a project id. This PR enables that, and does a small refactor of the HashDisplay component to allow showing the entire hash.

linear[bot] commented 6 months ago

ENG-831 Studio Web app should show me project ID somewhere

railway-app[bot] commented 6 months ago

This PR was not deployed automatically as @joewagner does not have access to the Railway project.

In order to get automatic PR deploys, please add @joewagner to your team on Railway.

joewagner commented 6 months ago

I'm moving this back to draft. I'm gonna look at updating the layout of the descriptions, and I found an issue where webpack doesn't like a div being a descendant of a p.

joewagner commented 6 months ago

I added an attempt to make the project page layout a bit more efficient.
Here are screengrabs for two different projects. One with a long description, and one with a short description. There's an image for each project at screen widths of 1200px, 800px, and 600px.

1200px Screen Shot 2024-03-20 at 10 22 33 AM Screen Shot 2024-03-20 at 10 22 44 AM

800px Screen Shot 2024-03-20 at 10 23 25 AM Screen Shot 2024-03-20 at 10 23 38 AM

600px Screen Shot 2024-03-20 at 10 24 10 AM Screen Shot 2024-03-20 at 10 24 25 AM

dtbuchholz commented 6 months ago

nice! one thing i just realized is we've had left margin on the project description and ID paragraphs. optionally, we could remove that so that it aligns with the header—i.e., it's currently set to ml-4, but i think it looks better w/o it:

Screenshot 2024-03-20 at 11 45 58 AM

...imo that's more natural, looks a bit weird to have it indented.

regardless, these improvements lgtm.

joewagner commented 6 months ago

@dtbuchholz good call on the margins. I might have messed them up when I was adding the responsive flexbox classes. Pushed a fix, I'll need another 👍

dtbuchholz commented 6 months ago

@joewagner they were always there, actually—they're on the current site!