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

Otter icon on the toolbar is too intrusive #1333

Closed
ineagu opened this issue Nov 17, 2022 · 13 comments · Fixed by #1378 or #1403
Closed

Otter icon on the toolbar is too intrusive #1333

ineagu opened this issue Nov 17, 2022 · 13 comments · Fixed by #1378 or #1403
Assignees
Labels
released Indicate that an issue has been resolved and released in a particular version of the product.

Comments

@ineagu
Copy link
Contributor

ineagu commented Nov 17, 2022

Description:

Besides bringing value to our users, I think we should do our best to don't create trouble. Personally, I find the Otter icon on the toolbar by default, not a good practice looking at the features that are listed there.

Export JSON - I think this should not even be enabled by default, how often and by what % of users is being used to justify the space?
Copy/ Paste Styles? - I think some users would use it heavily, others not

While Export, as I mentioned probably, can be off by default, the copy-paste styles I think it can go into the extended toolbar menu, vs on the main toolbar. As mentioned on the Form block review, the settings should be ideally ordered by usage.

Alternatives:

Do nothing.

@HardeepAsrani curious about your thoughts.

@HardeepAsrani
Copy link
Member

Will cc other people involved in this decision @JohnPixle @mghenciu @Soare-Robert-Daniel

Basically the idea was to make these features of Otter, like Copy/Paste, Sticky to Top, and so on to be more visible for users. Previously, we it was hidden in the Eclipses menu which was only easy to find, so most would just miss it if they didn't explore to see, plus that way also users wouldn't know that this feature was coming from Otter, not a Core feature.

Another issue here was that it wasn't possible to expose these features to Block-based Widgets screen as it didn't allow us to add those options to Eclipses menu.

@ineagu
Copy link
Contributor Author

ineagu commented Nov 17, 2022

That's a fair point, we can probably explore one by one those things.

  1. For giving exposure + choice a way is to have a full-screen onboarding when users can choose : Do you think you wanna use sticky bocks? Copy paste styles etc.
  2. We can leave the icon there as it is, but add a "Remove this icon" or "Move those settings to Eclipses Menu", which users can click and have them there.
  3. I still think that export json probably can be disabled by default and we can figure out how to make sure users is asked about.

@ineagu
Copy link
Contributor Author

ineagu commented Nov 17, 2022

I think the 2nd option solves all your concerns. There is exposure, easy way to find and work in block-based widgets. Just that now users would have a choice to easily move it away if they don't use a lot. Similar as you can re-order items on your iPhone.

@JohnPixle
Copy link
Contributor

I personally think that the Otter icon helps in the overall UX and discoverability of the features, and also allows users to have a clear mental grouping of Otter-related options.

It also helps the user understand which of the features are by Otter, and which are from core.

If you'd see Copy Styles for example under the Ellipsis menu, how can you understand that it's an Otter feature and not a core one?

I would also suggest that we consider adding more sections of Otter-related settings under the Otter icon. For example, Dynamic values and links would definitely fit in there.

Screenshot 2022-11-17 at 1 46 53 PM

What @ineagu said about having an option to remove the icon altogether might be a solution. But I think that this option would make more sense in the plugin settings instead of the dropdown itself. I feel it's UIX gravity is too much to be squeezed in such a small area.

Let me know your thoughts.

@HardeepAsrani
Copy link
Member

We can't have Dynamic Value/Link thing there as it's a format and it needs to be inside the Format thing.

And yes, I agree, allowing users to remove it seems like a good idea. I can see some finding it a bit annoying if they don't use any of it.

@JohnPixle
Copy link
Contributor

We can't have Dynamic Value/Link thing there as it's a format and it needs to be inside the Format thing.

Thanks, was not aware of the limitation.

What you all think about having the "remove" option in the settings page? Text is just for sample.

Screenshot 2022-11-17 at 2 16 20 PM

@mghenciu
Copy link
Contributor

Valid suggestion @JohnPixle . And we can have that ON by default, right?
Or do you guys think it should be OFF by default.

Overall I think the 'truth' is somewhere in the middle. As we initially added the Icon to increase Otter features visibility, and I was preparing to say that other Block Plugins are also doing this. And yes, Stackable is also doing this, the difference is that we add Otter on almost every block on the page - ours, core, 3rd party (which can be annoying if you neve use it), while Stackable adds the Icon to their blocks only.

@JohnPixle
Copy link
Contributor

Yeah, that's true Mihai. Most of others plugins are doing it, but usually only on their blocks.

I suggest we have it toggled on by default.

@ineagu
Copy link
Contributor Author

ineagu commented Nov 17, 2022

okay, I think it is better than currently, but still not ideal in my opinion, since at this stage of the product the focus should be delivering the best ux we can, people don't care that we worked at stuff that we wanna highlight and don't know how :D .

@HardeepAsrani I was wondering if we can't have those in the right sidebar? where do we have the visibility/animation settings?

I see that if I make a block sticky, I get the setting in the right side for it, is hard to get my head around having those settings in the toolbar, and seems out of place.

Also, a potential idea for making it clear that those are otter settings is to have the otter before the setting ( here is after) , but you get the idea; https://vertis.d.pr/i/8boc5e .

@HardeepAsrani
Copy link
Member

Once you click in it, you get the Otter logo there so it's not an issue. Like copy/paste styles don't make much sense in the sidebar. To make it easier, we can have a toggle in the Otter Menu itself to make it easier for user maybe?

@ineagu
Copy link
Contributor Author

ineagu commented Nov 17, 2022 via email

@JohnPixle
Copy link
Contributor

To briefly comment on some of your points:

Ionut said:

I was wondering if we can't have those in the right sidebar? where do we have the visibility/animation settings?

I will agree with Hardeep that the copy / paste will make more sense in the block toolbar rather than in the settings sidebar.


Hardeep said:

To make it easier, we can have a toggle in the Otter Menu itself to make it easier for user maybe?

I would not advise that, unless you are talking about the Otter settings sidebar. In this case yes, we can have the toggle there instead of the plugin settings, It's even more accessible. This sis where I mean:

Screenshot 2022-11-18 at 11 28 36 AM

Ionut said:

We can probably wait until things are more standized, since that toolbar can become a mess if 5 other plugins would do the same thing.

👍🏻 this makes sense too. I think it is a good idea not to rush with fixing this, as it's highly possible that the overall consensus regarding the toolbar might change in the future versions.

@Soare-Robert-Daniel Soare-Robert-Daniel linked a pull request Nov 25, 2022 that will close this issue
7 tasks
@Soare-Robert-Daniel Soare-Robert-Daniel removed a link to a pull request Dec 12, 2022
7 tasks
@Soare-Robert-Daniel Soare-Robert-Daniel linked a pull request Dec 12, 2022 that will close this issue
7 tasks
@HardeepAsrani HardeepAsrani mentioned this issue Dec 26, 2022
7 tasks
@HardeepAsrani HardeepAsrani linked a pull request Dec 26, 2022 that will close this issue
7 tasks
@HardeepAsrani HardeepAsrani mentioned this issue Jan 9, 2023
7 tasks
@pirate-bot
Copy link
Contributor

🎉 This issue has been resolved in version 2.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@pirate-bot pirate-bot added the released Indicate that an issue has been resolved and released in a particular version of the product. label Jan 10, 2023
@HardeepAsrani HardeepAsrani removed this from the 2.x milestone Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Indicate that an issue has been resolved and released in a particular version of the product.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants