Search Ideas

That means duplicate functionality; anytime I customize Devise in the future, I’ll have to remember to adjust this one method as well.

I could extract discussions#show into a new, separate StreamController or something like it. That controller would not use Devise.

Then again, I’d want to redirect users to the sign-in page (and then ideally back to where they were trying to go).

Duplicate of #2677/#2683.

While ideas load, cmd +f won’t work.

I could render the first ~10 top-level ideas immediately and only render the rest as turbo frames off screen. By the time the user scrolls down, they should all be loaded.

Yes, it would be even worse than #2677 (see criticism #2681), where only top-level ideas were turbo frames.

Too many requests when there are enough top-level ideas.

That means cmd + f won’t always work.

For large discussions, wouldn’t that flood the server with requests?

This page used to take ~3.5 seconds to load. Now it renders within 600ms :)

The following commits should address this:

• 3af3966 Clarify in title that someone revised an idea (rather than originated idea)

  The HTML title now says ‘Idea x revised by…’

• 6c70cea Underneath idea, indicate that someone revised an idea (rather than submitted it)

  It says ‘Dennis Hackethal, 1 day ago’ for new ideas, ‘Dennis Hackethal revised 1 day ago’ for revisions

• d20d386 Explain that users can revise each others’ ideas

  As part of the alert on the revision page, when the user is about to revise someone else’s idea.

• c5748e3 Turn ‘revise’ link into ‘revise their idea’ when it’s someone else’s idea

  Underneath each idea.

• e0fbd41 List user under each revision in version history

  So that each version is clearly attributed to the corresponding user.

• 06d3241 List contributors at top of version history

  Comma-separated list to see all contributors at a glance. Eg see here

Changing the view logic so that the controller can stream each deeply nested idea separately sounds non-trivial. It’s not clear to me how to do that currently. It may be possible someday if I adjust Hiccdown to allow the use of enumerators or something like that.

I could use ActionController::Live to stream top-level ideas to the page one by one. Instant page load.

Complex, requires non-trivial scroll listeners correlated to deeply nested ideas.

Just saw this app that lets you play ambient sounds at home: https://x.com/mirdhaaakanksha/status/1983238682154021218

Idea: ‘The Second Renaissance’, ‘2nd Renaissance’, ‘2R’ for short.

Replacing a raw SQL query in Idea.tree with a standard ActiveRecord query solves this issue.

I’ve since been able to reproduce the issue after all. Running a raw SQL query in Idea.tree in combination with the inclusion of the Live module seems to mess with Rails’s reloader.

A slow developer experience will slow down all further development, including bug fixes and feature rollouts, which hurts UX as well.

… copying extra stuff above and below the box quote, and neither gave me the > sign.

Cannot reproduce, neither on iPad nor macOS.

A single new idea somewhere down the tree could invalidate the cache and slow things down again.

Initial page loads would still be slow for users.

To be clear, if you copy the entire box quote and paste it into a textarea, it will start with the > sign. I just double checked.

You’re saying you’d still want the > if you only copy/pasted part of the box quote, right?

Cache invalidation for user-based caching sounds like a nightmare.