search

maxhu08 / mtab

🌴 lightweight new tab page extension
https://get-mtab.vercel.app
MIT License
339 stars 11 forks source link

[Request] Improved default bookmarks type #30

Closed Kenste closed 2 months ago

Kenste commented 2 months ago

I have a couple points that i would love seeing:

  1. Change the look of the default type to the be consistent with the custom one. I.e. go from: 1 to: 2

  2. Use the name of the bookmark for improved distinction of different bookmarks, similar to how i edited it in the second image. And for the default bookmark.type use no bookmark.color i guess? The icon should be used from the webpage itself, although i am guessing it should do it and it is bugged for me in the first image (Firefox on Linux Nobara).

  3. And in #25 you asked if the bookmark folders should be interactive, which was not implemented. Interactive folders would allow for a clean new tab page with functionally many bookmarks on them. An example of how it could be done would be the Speed Dial in the Vivaldi Browser: Vivaldi Speed Dial Folder or be creative, its up to you :P

maxhu08 commented 2 months ago

Okay, I think I'm going to add another type like default-2 and make it like how you said, but I'm pretty busy at the moment so you'll probably have to wait a while.

maxhu08 commented 2 months ago

Okay I've got a decent prototype for it, I still need to add the back button for when you enter a folder and more, but I'm getting close.

https://github.com/user-attachments/assets/233ff45f-39e4-4a8c-b5b6-dad010a66efd

maxhu08 commented 2 months ago

This has been released, so I'm going to close this issue.

Kenste commented 2 months ago

The changes look awesome!

I did notice, though, that you didn’t end up using the bookmark names for distinction. Especially now with the folders, I still think showing the names could make things a lot clearer at a glance.

Was that a choice you made? Totally cool if so—this is your project, after all! Just wanted to throw it out there again in case it was something that got overlooked.

maxhu08 commented 2 months ago

I'm planning to add the option to show text soon.