Dennis Hackethal’s avatar

Dennis Hackethal

@dennis-hackethal·Member since June 2024

Badges

 User
Registered their account.
 Initiator
Started their first discussion.
 Engager
Participates in three or more discussions.
 Novice
Submitted their first idea.
 Beginner
Submitted their 10th idea.
 Intermediate
Submitted their 50th idea.
 Advanced
Submitted their 100th idea.
 Critic
 Private
 Lieutenant
 Captain
 Defender
 Shield
 Watchman
 Copy editor
Created their first revision.
 Assistant editor
Created their 10th revision.
 Associate editor
Created their 50th revision.
 Professional
Submitted their 500th idea.
 Deputy editor
Created their 100th revision.
 Colonel
 Master
Submitted their 1000th idea.
 Bulwark
 General

Activity

  Dennis Hackethal commented on criticism #2701.

Old ideas can pollute discussions. Like in this meta thread.

#2701·Dennis HackethalOP revised about 19 hours ago

Proposed solution: allow people to archive ideas. Maybe only their own.

  Dennis Hackethal addressed criticism #2708.

There could be periodic reminders and a review board to make it easier.

#2708·Dennis HackethalOP, about 18 hours ago

That could just annoy people and cause them to unsubscribe from emails.

  Dennis Hackethal addressed criticism #2706.

This could inadvertently hide dormant but important ideas.

#2706·Dennis HackethalOP, about 19 hours ago

Could do it only for ideas with pending criticisms. If they have pending criticisms, how important can they be? This has the added benefit of creating an incentive for proponents of ideas to address criticisms.

  Dennis Hackethal addressed criticism #2707.

People might just forget to do this.

#2707·Dennis HackethalOP, about 18 hours ago

There could be periodic reminders and a review board to make it easier.

  Dennis Hackethal criticized idea #2703.

Proposed solution: edit a discussion to hide top-level ideas. That way, discussion owners can hide ideas they no longer deem relevant.

For example, completed tasks in discussions used as issue trackers, like this Meta thread, could be hidden so they don’t pollute the thread.

There could be a button for users to reveal hidden ideas so nothing is lost or hidden dishonestly. And direct links to hidden ideas would continue to work.

#2703·Dennis HackethalOP, about 19 hours ago

People might just forget to do this.

  Dennis Hackethal criticized idea #2704.

Proposed solution: ideas could be archived automatically if they haven’t been revised or criticized in 90 days, say.

#2704·Dennis HackethalOP, about 19 hours ago

This could inadvertently hide dormant but important ideas.

  Dennis Hackethal criticized idea #2703.

Proposed solution: edit a discussion to hide top-level ideas. That way, discussion owners can hide ideas they no longer deem relevant.

For example, completed tasks in discussions used as issue trackers, like this Meta thread, could be hidden so they don’t pollute the thread.

There could be a button for users to reveal hidden ideas so nothing is lost or hidden dishonestly. And direct links to hidden ideas would continue to work.

#2703·Dennis HackethalOP, about 19 hours ago

This requires manual action. Could mean a lot of work depending on the discussion.

  Dennis Hackethal commented on criticism #2701.

Old ideas can pollute discussions. Like in this meta thread.

#2701·Dennis HackethalOP revised about 19 hours ago

Proposed solution: ideas could be archived automatically if they haven’t been revised or criticized in 90 days, say.

  Dennis Hackethal commented on criticism #2701.

Old ideas can pollute discussions. Like in this meta thread.

#2701·Dennis HackethalOP revised about 19 hours ago

Proposed solution: edit a discussion to hide top-level ideas. That way, discussion owners can hide ideas they no longer deem relevant.

For example, completed tasks in discussions used as issue trackers, like this Meta thread, could be hidden so they don’t pollute the thread.

There could be a button for users to reveal hidden ideas so nothing is lost or hidden dishonestly. And direct links to hidden ideas would continue to work.

  Dennis Hackethal revised criticism #2698.

Feature idea: edit a discussion to hide top-level ideas. That way, discussion owners can hide ideas they no longer deem relevant.

For example, completed tasks in discussions used as issue trackers, like this Meta thread, could be hidden so they don’t pollute the thread.

There could be a button for users to reveal hidden ideas so nothing is lost or hidden dishonestly. And direct links to hidden ideas would continue to work.

Old ideas can pollute discussions. Like in this meta thread.

  Dennis Hackethal addressed criticism #2573.

Fixed as of f7833c6.

#2573·Dennis HackethalOP, 7 days ago

I can still reproduce the issue by clicking on the button to collapse/expand an idea.

  Dennis Hackethal revised criticism #2697.

Feature idea: edit a discussion to hide ideas. That way, discussion owners can hide ideas they no longer deem relevant. (For example, completed tasks in discussions used as issue trackers, like this Meta thread.) There could be a button for users to reveal hidden ideas so nothing is lost of hidden dishonestly.

Feature idea: edit a discussion to hide top-level ideas. That way, discussion owners can hide ideas they no longer deem relevant.

For example, completed tasks in discussions used as issue trackers, like this Meta thread, could be hidden so they don’t pollute the thread.

There could be a button for users to reveal hidden ideas so nothing is lost or hidden dishonestly. And direct links to hidden ideas would continue to work.

  Dennis Hackethal submitted criticism #2697.

Feature idea: edit a discussion to hide ideas. That way, discussion owners can hide ideas they no longer deem relevant. (For example, completed tasks in discussions used as issue trackers, like this Meta thread.) There could be a button for users to reveal hidden ideas so nothing is lost of hidden dishonestly.

  Dennis Hackethal addressed criticism #2679.

cmd + f won’t work reliably.

#2679·Dennis HackethalOP, 2 days ago

This problem will surface rarely – users would have to hit cmd + f immediately upon opening the page. For most users, by the time they start typing, the page is already fully loaded. So this seems like a small price to pay in exchange for discussion pages that always render faster.

  Dennis Hackethal addressed criticism #2679.

cmd + f won’t work reliably.

#2679·Dennis HackethalOP, 2 days ago

Now that parts of the page are purposely and visually disabled (see #2694), users may not expect everything to be working 100% during a loading state.

  Dennis Hackethal addressed criticism #2671.

JS modules are always deferred and unusable until the page is fully loaded. As a result, comment buttons and gutters won’t work while ideas are still streaming onto the page.

#2671·Dennis HackethalOP, 2 days ago

I now purposely prevent interactions with buttons and gutters, and gray them out, until the page is fully loaded. So instead of broken hover effects and interactions, the user gets intentionally disabled elements, and this intentionality is communicated to them.

Once the page is fully loaded, buttons and gutters are enabled and visually restored.

Since the browser’s loading indicator remains visible until then, this behavior shouldn’t violate user expectation.

  Dennis Hackethal addressed criticism #2692.

I could override authenticate_user! in the DiscussionsController.

#2692·Dennis HackethalOP, 2 days ago

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

  Dennis Hackethal addressed criticism #2670.

Incompatible with Devise authentication: https://github.com/heartcombo/devise/issues/2332

#2670·Dennis HackethalOP, 2 days ago

I could override authenticate_user! in the DiscussionsController.

  Dennis Hackethal addressed criticism #2670.

Incompatible with Devise authentication: https://github.com/heartcombo/devise/issues/2332

#2670·Dennis HackethalOP, 2 days ago

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

  Dennis Hackethal addressed criticism #2688.

The thread suggests a workaround: use authenticated do … blocks in routes.rb instead of before_action :authenticate_user! in controllers.

It’s probably a good idea to do this anyway to avoid divulging the existence of routes that unauthenticated users don’t need to know exist. (They will get a 404 instead of a 401.)

#2688·Dennis HackethalOP revised 2 days ago

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

  Dennis Hackethal revised criticism #2686.

The thread suggests solution: use authenticated do … blocks in routes.rb instead of before_action :authenticate_user! in controllers.

It’s probably a good idea to do this anyway to avoid divulging the existence of routes that unauthenticated users don’t need to know exist. (They will get a 404 instead of a 401.)

The thread suggests a workaround: use authenticated do … blocks in routes.rb instead of before_action :authenticate_user! in controllers.

It’s probably a good idea to do this anyway to avoid divulging the existence of routes that unauthenticated users don’t need to know exist. (They will get a 404 instead of a 401.)

  Dennis Hackethal revised criticism #2673.

The thread contains a suggested solution: use authenticated do … blocks in routes.rb instead of before_action :authenticate_user! in controllers.

It’s probably a good idea to do this anyway to avoid divulging the existence of routes that unauthenticated users don’t need to know exist. (They will get a 404 instead of a 401.)

The thread suggests solution: use authenticated do … blocks in routes.rb instead of before_action :authenticate_user! in controllers.

It’s probably a good idea to do this anyway to avoid divulging the existence of routes that unauthenticated users don’t need to know exist. (They will get a 404 instead of a 401.)

  Dennis Hackethal criticized idea #2636.

On initial page load, I could just load the first ten or so top-level ideas and their immediate children, just to reduce wait times and populate the page. Then load the rest asynchronously.

#2636·Dennis HackethalOP, 5 days ago

Duplicate of #2677/#2683.

  Dennis Hackethal criticized idea #2677.

The top level ideas could be rendered as turbo frames of ideas#show.

#2677·Dennis HackethalOP, 2 days ago

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

  Dennis Hackethal addressed criticism #2680.

I just tried this. Seemed promising at first but sometimes ideas load out of order. Looks horrible.

#2680·Dennis HackethalOP, 2 days ago

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.