Conversation
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Deploying matrix-website with
|
| Latest commit: |
b0786f9
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://35981ec4.matrix-website.pages.dev |
| Branch Preview URL: | https://mtrnord-get-started.matrix-website.pages.dev |
HarHarLinks
left a comment
There was a problem hiding this comment.
looks very promising to me!
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
This requires a followup change later to also display this on the ecosystem. This is out of scope at this time Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
This comment was marked as resolved.
This comment was marked as resolved.
awtj8o81ryywg793
left a comment
There was a problem hiding this comment.
Alongside the points raised in this review, I've noticed that the "Get an app" table isn't responsive.
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
|
This is an amazing redesign! Thanks for tackling it, the Try Matrix page has needed some love for a long time, and this redesign is made tastefully and efficiently! Simple instructionsOn desktop, I like that the simple "Get Element X and create an account on matrix.org" instructions take the whole. It keeps things simple for people who just want to give it a go. On mobile, the instructions are slightly larger than the screen as seen from an iPhone 11 Pro, a reasonably not too old device. Fiddling with the font-size, I can make the instructions fit in. Maybe it would make sense to Clients listSeeing all the featured clients in a simple and compact list is really great. It's nicely presented and less overwhelming than the regular view we have on /ecosystem/clients. For the screenshots, I think we should enforce a specific aspect ratio. 4:3 seems to be an interesting one. For desktop, it's seamless. For mobile, we can ask to have a collage of 2 or 3 screenshots in the same image to achieve the same aspect ratio. I think it could be interesting to have an anchor for each client as well, so it's possible to link to Bells & whistlesTwo lines of CSS could go a long way:
|
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
| <label for="pf-linux" class="platform-filter-btn">Linux</label> | ||
| </div> | ||
|
|
||
| <div class="client-cards"> |
There was a problem hiding this comment.
the client cards and their text is surprisingly small on mobile, and partially also on desktop
There was a problem hiding this comment.
Need more details here I think. Not sure what exactly you mean. It feels fine on my side
There was a problem hiding this comment.
The font size seems to change somewhat arbitrarily all over the place, and there is even some inconsistency (e.g. between "good for" text and the m.org homeserver details.
Maybe let's take a step back:
- What is the goal?
- Can it be ~1 consistent size?
- (Should text in general be larger on m.org?)
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
…on mobile Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
… lightbox for some clients multiple screenshots make sense. The parent is limited to 4:3. Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
197b92e to
6f62643
Compare
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Thats fixed by using clamp now so it doesnt overflow as likely. I also centered the text within now. |
|
Note: Missing/blocking is that I also add the missing screenshots. |
weeman1337
left a comment
There was a problem hiding this comment.
I like the new TL;DR
Other than that only some smaller notes.
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
|
can we get more anchors so i can send people e.g. to the pick a server section? |
|
|
||
| // ── Account tiers (Create an account section) ───────────────────────────── | ||
|
|
||
| .account-tiers { |
| } | ||
| } | ||
|
|
||
| .scroll-down-indicator { |
There was a problem hiding this comment.
The no download needed text is also BS on mobile.
Signed-off-by: MTRNord <MTRNord@users.noreply.github.com>
|
@MTRNord This is a great PR already and I wouldn't want to let it bitrot, do you have some bandwidth to tackle it or would you rather someone else takes over :) ? |
I would love if someone can help with the client screenshots. I am kinda lacking motivation for that currently :/ So yes help appreciated! |
|
(https://github.com/AlexanderVanhee/Gradia looks like it might create very pretty screenshots) |



Description
Well its a whole redesign. I suggest having a look at the deployment for all the things. Trying is better than words in this case
Related issues
Role
Website and Content WG
Timeline
Whenever
Signoff
See commits