Rotonde / rotonde-client

Rotonde Base Client
https://client-neauoire.hashbase.io/
MIT License
227 stars 49 forks source link

Merge rotonde-mod 0.1.3 with rotonde-client #106

Closed 0x0ade closed 6 years ago

0x0ade commented 6 years ago

I honestly don't expect this PR to be merged as-is, but hope that at least parts of it will find their way into rotonde-client.

This will hurt a little because

The good news, though, is that the pagination and the related element reuse changes reduced the crash rate. We're not manipulating the DOM via innerHTML as aggressively anymore. I assume Beaker 0.8 will just deal with DOM updates better?

This adds the following features:

image

image

image

image

I didn't include .ext#URL/. media support because it requires /media/content/.ext/index.html redirecting to window.location.hash for backwards-compatibility and because it allows non-self-hosted media.

neauoire commented 6 years ago

That is a lot of changes, I want it all, could you fix the css conflicts and I ll merge. You seem to have respected the current coding style. I think this is a pretty sold build. I don't see why I wouldn't merge as-in and build upon 👍

deanputney commented 6 years ago

😍

0x0ade commented 6 years ago

Thanks for merging it so quickly and for fixing the css issues! It's 00:25 here in Germany and I can't fall asleep anymore - am too excited to boot up my PC after uni tomorrow :D

I didn't want this to be merged "prematurely" because I didn't want to introduce any bugs without others testing it (had to fix some bugs after the initial -mod "merge").
It's great to hear that everything's working as it should :)

Again, thank you for rotonde!

dcposch commented 6 years ago

@neauoire @0x0ade i think this did break a few things

image

i'll see if i can fix it, but no promises

neauoire commented 6 years ago

I fixed that, I'm working on making sure the tabs's width don't shift, and that the feed displays on load. There are still few things before we can start seeding this, but overall, well worth it 👍

eelfroth commented 6 years ago

whoa, that's quite a handful — great job! the message counter seems to work now, and I do like your way of counting them in Feed.refresh. this is even better than what I had in mind :+1: