~cadence

https://cadence.moe

~cadence/bibliogram-devel

Last active a day ago

~cadence/tube-devel

Last active a day ago

~cadence/bibliogram-announce

Last active a month ago

~cadence/public-inbox

Last active 3 months ago

~cadence/bliz-devel

Last active 3 months ago

~cadence/tube-announce

Last active 3 months ago

~cadence/carbon-discuss

Last active 3 months ago

~cadence/bibliogram-translate

Last active 4 months ago

~cadence/bliz-announce

Last active 4 months ago
View more

Recent activity

Re: [PATCH cloudtube] Autofocus lower searchbar on homepage a day ago

From Cadence Ember to ~cadence/tube-devel

Resolved in 8486517 using user-agent sniffing. Not perfect, but it
should cover the _vast_ majority of cases.

Re: [PATCH cloudtube] Proxy captions via new /proxy route a day ago

From Cadence Ember to ~cadence/tube-devel

> +			let remoteURL = url.searchParams.get("url")

To aid readability, let's use `const`. Naming the variables, e.g.,
`path` instead of `URL` would also help, when it is a path.

> +					statusCode: 401,
> +					content: "Unauthorized"

You need a `contentType`. Would `application/json` be appropriate?
`text/plain` would be acceptable too. Might be useful to make the
message something like `CloudTube: Unauthorised` to indicate to humans
that this response didn't hit the proxy. Otherwise it would be
ambiguous whether CT or NL gave this response.

Re: [PATCH cloudtube v2 2/2] Implement video captions 6 days ago

From Cadence Ember to ~cadence/tube-devel

Thanks, I got this all sorted and working. I hacked at your commits a
bit to fix up a couple of issues, change the route, and add
auto-caption stuff. Hope you don't mind!

[PATCH cloudtube] Autofocus lower searchbar on homepage 7 days ago

From Cadence Ember to ~cadence/tube-devel

I agree that this would be useful for computers, but some mobile devices, 
when there is an autofocus field, will also bring up the keyboard 
automatically, which I find disorientating.

Ideally I'd like it to autofocus for computers only, and I'm not sure how 
I'd do that.

Cadence

Re: [PATCH] Update Turkish translation 18 days ago

From Cadence Ember to ~cadence/bibliogram-devel

Thanks for this. Your patch didn't apply cleanly, but I believe I have
made the required changes by hand. Let me know if it's all correct!

Re: [PATCH v2] Removed default config file from Dockerfile, added .git folder to dockerignore 22 days ago

From Cadence Ember to ~cadence/tube-devel

Okay, wow, that's really interesting and I do agree with those decisions. 
I'll give this a check to see how it works in practice, then I think this 
is good to be merged.

Re: [PATCH v2] Removed default config file from Dockerfile, added .git folder to dockerignore 23 days ago

From Cadence Ember to ~cadence/tube-devel

Hi, I'm finally getting to look at this. Thank you so much for your
patience.

> +/configuration.py

Did you mean to put the main configuration file in dockerignore?

> +COPY ./requirements.txt /workdir/requirements.txt
> +
> +RUN pip install -r requirements.txt
> +
> +COPY . /workdir

Is there a reason why you don't `COPY .` straight away? Why 2 copy

[PATCH] Captions support 27 days ago

From Cadence Ember to ~cadence/tube-devel

Thanks so much for this! Tomorrow I can properly review it, but taking a 
peek at the code now, there's a couple of things that stand out to me:

- kind should be `captions` since it is a transcription of audio, not a 
translation; see 
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/track
- should put the result of `getSettingsOrDefaults` in a variable to make 
it easier to extract other settings in the future
- should validate the `lang` query parameter somehow

I can deal with these notes on my own tomorrow, as well as doing my full 
review, but if you want make these changes yourself, you're free to 
submit an updated patch.