Create toggle for logo #38
Conversation
|
Hi @jcorbalanm, thanks for your PR! Interesting idea, do you dislike the logo? |
Fivefold
left a comment
Fivefold
left a comment
There was a problem hiding this comment.
Good job, small but clean PR!
| </label> | ||
| <div class="form-input-hint"> | ||
| Shows or hides the extension logo on the results title. | ||
| </div> |
There was a problem hiding this comment.
I would try to order the options from most important to least important. Personally, I think hiding the logo is something few people will need, so I would put it last in the advanced settings. Feel free to share your thoughts if this is a bad idea.
There was a problem hiding this comment.
I didn't know where to put it. Putting it under "Default open link type" seems good idea. I don't know how many people change the Theme of injection box, but obviously we can put it at the bottom, I have no problem with that.
There was a problem hiding this comment.
Option at the bottom of advanced (to me seems a bit buried and may not be seen):
Option in the middle of advanced:
@Fivefold I don't necessarily dislike it but i prefer a more minimalist look on my search results. I made this PR pretty much for myself because I found it easy. |
2 participants
Small change to enable and disable logo visibility (enabled by default). Tested.