nyxt icon indicating copy to clipboard operation
nyxt copied to clipboard

Rename `marks` to `mark`?

Open aadcg opened this issue 1 year ago • 4 comments

#2520 introduced marks-actions, which adds to return-actions and selection-actions.

However, marks-actions sounds really weird and it's hard to pronounce.

I'd rename marks to mark. Indeed, selection and mark are radically different since the former is always a singleton but the latter isn't. They're actually both lists, but for very different reasons related to the way they're implemented.

It doesn't sound odd to me that mark would a be concept defined by a word in the singular form that refers to a set of entities.

aadcg avatar Aug 29 '22 13:08 aadcg

On second thoughts, I'd suggest something even more radical.

We have the following concepts: suggestions, selection and marks.

Wouldn't it be saner to have suggestions, current-suggestion and selection?

Suggestions are the prompt options; current-suggestion is a singleton of suggestions; and selection is a subset of suggestions.

aadcg avatar Aug 29 '22 13:08 aadcg

I guess the initial naming comes from Helm. If the new naming is drains less cognitive resources, then why not :)

I'll leave that to @Ambrevar, though, as he wanted to abstract promtper into a separate library, and that is likely when the renaming should happen.

aartaka avatar Aug 29 '22 14:08 aartaka

Yes, the naming comes from Helm. "Mark" is not very familiar outside of this context, so yes, I'm in favour of André's renaming suggestion.

Shall it wait for 4.0 though? We've already got a lot on our hands...

Ambrevar avatar Sep 01 '22 10:09 Ambrevar

Yes, this is not particularly important.

aadcg avatar Sep 05 '22 20:09 aadcg