Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inserter: sort blocks by recency #888

Closed
jasmussen opened this issue May 24, 2017 · 7 comments
Closed

Inserter: sort blocks by recency #888

jasmussen opened this issue May 24, 2017 · 7 comments
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface [Type] Task Issues or PRs that have been broken down into an individual action to take
Milestone

Comments

@jasmussen
Copy link
Contributor

Should we replace the "Common" category with a "Recently used" area?

Or should there be a "Recently used" category at the top, in addition to the "Common" category? Perhaps it only shows 4 blocks, or something in that vein?

@jasmussen jasmussen added Design [Feature] Inserter The main way to insert blocks using the + button in the editing interface [Type] Question Questions about the design or development of the editor. labels May 24, 2017
@karmatosed
Copy link
Member

To me, recently used makes more sense so +1.

@BE-Webdesign
Copy link
Contributor

Recently used in the context of that post? Doing that across posts would be a significant amount trickier to do from a technical perspective.

@jasmussen
Copy link
Contributor Author

Recently used in the context of that post? Doing that across posts would be a significant amount trickier to do from a technical perspective.

No, I'd think recently used by the user across posts. So if all you ever do is use text, then that block is the first one, whereas if you use the "Cover Image" all the time, that would climb up the list.

@BE-Webdesign
Copy link
Contributor

No, I'd think recently used by the user across posts. So if all you ever do is use text, then that block is the first one, whereas if you use the "Cover Image" all the time, that would climb up the list.

This would be pretty technically challenging depending on how far it would be taken, but probably a great user experience.

@folletto
Copy link
Contributor

I'd suggest to use not linear recency (macOS emojis inserter) but weighted recency (Slack emojis inserter).

Difference:

  • linear recency: any time an emoji gets used jumps at the top of the recency list.
  • weighted recency: any time an emoji gets used it gets a +1 counter. The top X emojis by total counter value gets at the top of the list.

The benefits of the weighted one is that linear recency can get wiped out quickly with a single time that one does something different, while weighted is more stable over time and far far far more likely to have the actual ones that the user uses.

The weighted recency is a bit more complex, so we can start with linear at first but wrapping it in a function cleanly so we can easily replace it later with weighted.

A way to approach weighted recency that is relatively simple is to store them locally, so while yes, every browser / device will get a different list, it's still beneficial as when it starts it still behaves like linear recency. But it's far far simpler than storing them somewhere in the DB.

@mtias mtias added [Type] Task Issues or PRs that have been broken down into an individual action to take and removed [Type] Question Questions about the design or development of the editor. labels Jun 22, 2017
@notnownikki
Copy link
Member

I have the start of this working at #1506

@notnownikki
Copy link
Member

#1506 does a "Recently used" category at the top. The majority of the work in it was keeping tab working when blocks are in more than one category, but the code is easy to change to communicate with an API to do weighting etc.

it might need some tweaking with the search bar move.

@mtias mtias added this to the Beta 3 milestone Jun 29, 2017
@mtias mtias changed the title Inserter: Consider sorting blocks by recency Inserter: sort blocks by recency Jun 29, 2017
@mtias mtias modified the milestones: Beta 0.5.0, Beta 0.4.0 Jul 8, 2017
@mtias mtias mentioned this issue Jul 11, 2017
@mtias mtias closed this as completed Jul 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Inserter The main way to insert blocks using the + button in the editing interface [Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

No branches or pull requests

6 participants