gearsdigital / enhanced-toolbar-link-dialog

Extend Kirby's default link dialog with search functionality, allowing you to easily create links to existing or external pages at your fingertips.
MIT License
62 stars 8 forks source link

Feature request: Option to disable the anchor field #82

Closed arnaudjuracek closed 1 year ago

arnaudjuracek commented 2 years ago

First of all, thank you for your plugin. It is so simple that I may use it by default on all my new Kirby projects, until it ends up being eventually integrated to the Kirby core.

It would be really nice to have a config option to disable the anchor field, to allow building simpler websites for unskilled clients who do not need to know what an anchor is.

gearsdigital commented 2 years ago

You are very welcome @arnaudjuracek! Glad you find my plugin useful 🙏🏻

I like your idea and I'm going to implement this not only for the anchor, but also for every other field. It's a bit more challenging to keep the layout consistent. Not every field takes the full width, but I'm confident to find a proper solution :)

gearsdigital commented 1 year ago

@arnaudjuracek Before I'm going to start to work on this issue I need to clarify what do you mean by disable.

  1. The field is visible to the user, but disabled, so that he can't add any input Bildschirmfoto am 2023-04-14 um 09 26 44

  2. The field is invisible to the user and removed completely Bildschirmfoto am 2023-04-14 um 09 27 13

arnaudjuracek commented 1 year ago

IMHO the UX is better for the user when the field is completely hidden, but you may have other scenarios in mind where disabling the field makes more sense ?

Thanks for your time.

gearsdigital commented 1 year ago

Okay, let's do both ^^ The user can decide whether he wants to disable or hide a field. Thanks for your input!

gearsdigital commented 1 year ago

:tada: This issue has been resolved in version 3.2.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

gearsdigital commented 1 year ago

@arnaudjuracek Let me know if this feature works for you :)