Skip to content
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

Trial comets #1794

Merged
merged 8 commits into from
Sep 23, 2022
Merged

Trial comets #1794

merged 8 commits into from
Sep 23, 2022

Conversation

pkova
Copy link
Contributor

@pkova pkova commented Sep 12, 2022

Adds a page for accessing trial comets /trial, and a static page where I can redirect people whose trials have ended /trial-over. Needs copy and possibly styling on both pages, backend is now good to go. Also, nothing links the trial landing page so we need to figure out where to put it. @gordoncc

@matildepark
Copy link
Collaborator

There's something about this and #1784 that are currently not served in our websites — they're effectively app interfaces, not web pages, and I think they need their own sort of layout to them. I've tried to gesture at these in the past by making them "card-like", wrapping the whole thing but, I don't know, just calling attention to it.

The front-end looks pretty good! Definitely needs at least copy; should also be tied into getting-started section.

@matildepark
Copy link
Collaborator

Er, as something me and Gordon can do I mean.

@gordoncc
Copy link
Contributor

gordoncc commented Sep 13, 2022

Put together some designs here: https://www.figma.com/file/rlQkryJk1WjzTnMNZzsAkl/Urbit.org?node-id=304%3A994

@matildepark - think you can sort this?

Trial Concluded
Explore the Network - Not Available
Explore the Network
Index

@matildepark
Copy link
Collaborator

Wow, synchronicity ...

@gordoncc
Copy link
Contributor

Looking good, just noticing that the button kinda breaks at smaller screen widths

Screen Shot 2022-09-13 at 10 36 42 AM

@gordoncc
Copy link
Contributor

One more feature request – can we turn the image into a carousel that rotates through images every 2s? I feel like we need to do a little more showing on this page.
Explore the Network

@gordoncc
Copy link
Contributor

Here are some carousel images:

Carousel 1

Carousel 2

Carousel 3

Carousel 4

@matildepark
Copy link
Collaborator

What if we just used the video from the discord page?

@gordoncc
Copy link
Contributor

Oh, even better idea. Let's do that.

@matildepark
Copy link
Collaborator

It's just like, to me, a carousel is like "ah, all these memories of Landscape .... they will be lost in time, like tears in rain ..."

@gordoncc
Copy link
Contributor

gordoncc commented Sep 13, 2022 via email

@gordoncc
Copy link
Contributor

Flexbox is still having issues in Chrome

Screen Shot 2022-09-13 at 12 41 44 PM

@gordoncc
Copy link
Contributor

And the button /text should be left aligned rather than middle aligned (safari)

Screen Shot 2022-09-13 at 12 42 42 PM

@gordoncc
Copy link
Contributor

Ok, looking much better. I made a new video that I think is a bit of a better demo, why don't we use this one: https://storage.googleapis.com/media.urbit.org/site/explorethenetwork.mp4

@matildepark
Copy link
Collaborator

I guess we're good to deploy. Who has final stamp? @gordoncc @pkova ?

@pkova
Copy link
Contributor Author

pkova commented Sep 14, 2022

We ran some numbers with @jalehman and decided to do a lot more comets per day but for just 24h. I'll make the necessary changes on the backend, fix the copy to say 24h instead of one week and then merge.

@matildepark
Copy link
Collaborator

A day is brief; that’s a lot of comet turnaround. Can we confirm the subscription overhead it’ll add to bigger groups with an Ames / userspace person?

@pkova
Copy link
Contributor Author

pkova commented Sep 14, 2022

I will do moons and breach them at the end of the trial period.

@pkova
Copy link
Contributor Author

pkova commented Sep 23, 2022

And we are live. Fear the rosmyn^fordet.

@matildepark matildepark deleted the pkova/shore branch November 15, 2022 21:35
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.

3 participants