Skip to content

Add Tooltip component #810

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

Conversation

Komediruzecki
Copy link
Contributor

@Komediruzecki Komediruzecki commented Feb 23, 2021

Add Tooltip component (#552)

  • Add tooltip component
  • Add basic portal component for tooltip
  • Add tooltip component in navigator buttons
  • Add tooltip for button components
  • Add translation for button tooltip texts
  • Add tooltip delay
  • Show tooltip arrow on edges at the correct location
  • Fix tooltip arrow center position issue

Few examples of how it looks:
TooltipWorkspaceNewNoteButton
TooltipOpenContextView
TooltipPathTooltip
TooltipAddLabelSolarizedDark
TooltipEditTitleSolarizedDark
TooltipIndentOptionsBottomBar

I left the coloring and styling to whoever can make it work and used tooltip background color available in the theme.
In this (blackish) color arrow is sometimes not seen so clearly and on a dark background, it would also be a bit harder to spot. Meaning separate colors for each theme (black/white/sepia) could be used, an example of that would be something like this:

TooltipNavigateToFolder
TooltipShowMdPreviewSepia

The non-transparent color would also help I think.

Tested in

  • Local dev
  • Production AppImage (Linux)

@Komediruzecki Komediruzecki added the awaiting review ❇️ Pull request is awaiting a review. label Feb 23, 2021
Add tooltip component
Add basic portal component for tooltip
Add tooltip component in navigator buttons
Add tooltip for button components
Add translation for button tooltip texts
Add tooltip delay
Show tooltip arrow on edges at correct location
Fix tooltip arrow center position issue
@Komediruzecki Komediruzecki force-pushed the feature/improve-tooltip-handling branch from d4e9275 to a2881c7 Compare March 16, 2021 21:30
@Komediruzecki
Copy link
Contributor Author

@Rokt33r we could use this code to improve cloud tooltips, when adding tooltip alongside edge of screen, the whole tooltip can not be viewed, Maybe I should rebase this PR and add improvements to shared tooltip component and merge this instead?

@Komediruzecki
Copy link
Contributor Author

Closing in favor of new local space only repository, will be reopened and rebased there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review ❇️ Pull request is awaiting a review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant