Veritula – Meta

Showing only #3950 and its comments.

See full discussion

Log in or sign up to participate in this discussion.

With an account, you can revise, criticize, and comment on ideas.

Discussions can branch out indefinitely. Zoom out for the bird’s-eye view.

Dennis HackethalOP·#3950

Code blocks need syntax highlighting.

Veritula used to have this feature but I removed it when diffing changed.

Criticism

Dennis HackethalOP·#3951

Done as of cc1ab95.

Ruby example:

1
2
3
4
5
6
7
8
9
10
11
ruby
def criticized? idea
  pending_criticisms(idea).any?
end

def pending_criticisms idea
  criticisms(idea).filter { |c| pending_criticisms(c).none? }
end

def criticisms idea
  children(idea).filter(&:criticism?)
end

JS example (h/t ChatGPT):

1
2
3
4
5
6
7
8
9
10
11
javascript
function criticized(idea) {
  return pendingCriticisms(idea).length > 0;
}

function pendingCriticisms(idea) {
  return criticisms(idea).filter(c => pendingCriticisms(c).length === 0);
}

function criticisms(idea) {
  return children(idea).filter(c => c.isCriticism);
}

Criticism of #3950Criticized3

Dennis HackethalOP·#3956

2nd of 2 versions

When an idea has nothing but a code block, there’s too much of a margin at the bottom, between the block and the border of the highlight.

Criticism of #3951Criticized1

Dennis HackethalOP·#3954

Fixed as of a44c6c0.

Criticism of #3956

Dennis HackethalOP·#4024

2nd of 2 versions

The diff view can’t handle the removal/replacement of entire code blocks yet. The removed block looks broken, the new block doesn’t show at all. See activity 3207 in dev.

Criticism of #3951

Dennis HackethalOP·#4013

There’s a small issue related to previewing changes in code blocks: even when there are no changes yet, if the code overflows horizontally, the scroll shadow is shown through DOM manipulation, which in turn triggers the diffing library into thinking the user made a change.

So then the same code block is shown without any changes, under the ‘Changes’ tab, which is confusing. It should still just say ‘No changes’.

Criticism of #3951

Dennis HackethalOP·#4016

There’s an issue with horizontal scroll for overflowing code blocks in the activity feed on mobile. Can’t scroll all the way to the right.

Criticism of #3951Criticized1

Dennis HackethalOP·#4022

Fixed as of e49cd8d.

Criticism of #4016

Dennis HackethalOP·#4023

Would be nice if the copy button was sticky-top so that it scrolled with the user.

Criticism of #3951

Dennis HackethalOP·#4032

Done as of 43c4ecc.

Criticism of #4023Criticized1

Dennis HackethalOP·#4033

I spoke to soon. Rolling this back for now. Too jittery when scrolling on mobile. Non-trivial to implement. Need to see how other sites do it.

Criticism of #4032

Dennis HackethalOP·#4027

2nd of 2 versions

On mobile, there needs to be more of a padding on the right, inside the code block.

Criticism of #3951Criticized1

Dennis HackethalOP·#4029

Done as of 609b5c3.

Criticism of #4027

Dennis HackethalOP·#3952

When the code overflows horizontally, a subtle inset shadow on the side shows that you can scroll:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
javascript
const posts = [
  {id: 1, title: "Understanding JavaScript Closures in Depth", url: "https://example.com/articles/javascript-closures-deep-dive"},
  {id: 2, title: "A Complete Guide to Modern Web Development Practices", url: "https://example.com/articles/modern-web-dev-guide"},
  {id: 3, title: "Exploring the Node.js Event Loop and Async Patterns", url: "https://example.com/articles/nodejs-event-loop"}
];

function formatPost(post) {
  return `${post.id}: ${post.title} -> ${post.url}`;
}

function prettyPrint(posts) {
  return posts.map(formatPost).join(" | ");
}

console.log(prettyPrint(posts));