cadence · they/them
cadence pushed to rich-messages at cadence/Carbon 2020-11-05 03:32:53 +00:00
714147b980 Fix lazy loading cache
cadence pushed to princess at cadence/Carbon 2020-11-05 03:24:13 +00:00
ebf6e7ea78 Show proper user data in room list
cadence commented on issue cadence/Carbon#28 2020-11-02 08:13:01 +00:00
Not redirecting to login page on invalid token

Yeah, this is known. It'll probably be addressed when the site structure is changed to have a homepage, docs pages, and a better way of navigating around everything.

cadence commented on pull request cadence/Carbon#20 2020-11-01 23:02:35 +00:00
WIP: Some visual improvements for login

No, px will not cause issues.

cadence suggested changes for cadence/Carbon#20 2020-11-01 09:16:07 +00:00
WIP: Some visual improvements for login

Just tried this out and I like it a lot actually! There's a couple of nitpicks before we can show it to everyone.

cadence commented on pull request cadence/Carbon#20 2020-11-01 09:16:07 +00:00
WIP: Some visual improvements for login

12px please, it doesn't need to be tiny.

cadence commented on pull request cadence/Carbon#20 2020-11-01 09:16:07 +00:00
WIP: Some visual improvements for login

This needs to be a lighter red for better contrast. Try #ff6561 I guess?

cadence commented on pull request cadence/Carbon#20 2020-11-01 09:16:07 +00:00
WIP: Some visual improvements for login

This link doesn't go anywhere. I assume the site is yet to be created?

cadence commented on pull request cadence/Carbon#20 2020-11-01 09:16:07 +00:00
WIP: Some visual improvements for login

style: indentation 2 spaces, line break above selectors.

cadence commented on pull request cadence/Carbon#20 2020-11-01 09:16:07 +00:00
WIP: Some visual improvements for login

Please create a class .homeserver-question and target that instead. Using IDs in CSS sounds fine but quickly turns into hell. I know this from experience.

cadence commented on pull request cadence/Carbon#20 2020-11-01 09:16:07 +00:00
WIP: Some visual improvements for login

This does nothing, remove it.

cadence commented on pull request cadence/Carbon#24 2020-10-30 23:49:15 +00:00
Rich message rendering

1 MB bundle size is kind of ridiculous. We could simply not highlight code blocks, or we could maybe inline async import the module on demand when we actually need to do highlighting, so it's out of…

cadence commented on issue cadence/Carbon#18 2020-10-30 11:51:06 +00:00
[Discussion] New user UX

That's a really good prompt message.

I'd like to use a circled ? icon next to the "homeserver" label that responds to clicking, rather than hovering, or requiring the input box to be empty. Those…

cadence commented on pull request cadence/Carbon#24 2020-10-29 07:46:58 +00:00
Rich message rendering

Again here. I think a better way would be to have a base class like MembershipEvent, and then for each kind of membership we subclass it and only implement the canRender method on each.

cadence commented on pull request cadence/Carbon#24 2020-10-29 07:46:58 +00:00
Rich message rendering

Just going through a visual analysis of your code, I'll test it and give a full review in the future.

cadence commented on pull request cadence/Carbon#24 2020-10-29 07:46:58 +00:00
Rich message rendering

style: can we put spaces after // ?

cadence commented on pull request cadence/Carbon#24 2020-10-29 07:46:58 +00:00
Rich message rendering

The original idea here was that we'd have a predicate for things like whether the method can be displayed as a message group, whether it should show a timestamp, or whatever other things could affect the display based on the class. I haven't actually put this method to any use yet.

cadence commented on pull request cadence/Carbon#24 2020-10-29 07:46:58 +00:00
Rich message rendering

Not sure how I feel about a function that creates classes. Can we inherit from a base class instead?

cadence pushed to princess at cadence/Carbon 2020-10-29 04:32:18 +00:00
ff196a64bb Improve message sender rendering
a4c7f29ec9 Emacs files to gitignore
5bfe98bdf4 Stop scrollback at top of timeline
Compare 3 commits »
cadence pushed to princess at cadence/Carbon 2020-10-26 10:57:05 +00:00
f4b368ea3e Better message line breaking