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

Add tooltips to the note detail toolbar #599

Merged
merged 9 commits into from
Sep 3, 2020

Conversation

laudep
Copy link
Contributor

@laudep laudep commented Aug 27, 2020

See #552

  • New English texts for tooltips
  • Basic tooltips on the note detail toolbar component and its children
    • NoteDetailToolbar
      • NoteDetailFolderNavigator
      • NoteDetailTagNavigator
        • TagNavigatorListItem
      • ToolBarIconButton

}: NoteDetailFolderNavigatorProps) => {
interface FolderNavItemProps {
path: string
children?: React.ReactNode
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should not accept children

noteFolderPathname,
}: NoteDetailFolderNavigatorProps) => {
interface FolderNavItemProps {
path: string
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

storageId, storageName, noteId, noteFolderPathname are already resolved from parent component. We don't need to parse again. It will make our app slower.

? getFolderTooltip(folderName)
: getStorageTooltip(storageName)

const isActive =
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

active

return (
<FolderNavItemButton
title={tooltip}
href={path}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is a button, not an anchor element


const tooltip = folderName
? getFolderTooltip(folderName)
: getStorageTooltip(storageName)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is a folder nav item, not a storage item.

>
{storageName}
</FolderNavItem>
<FolderNavItem path={`/app/storages/${storageId}/notes/${noteId}`} />
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to introduce another component

@Rokt33r
Copy link
Member

Rokt33r commented Sep 2, 2020

I'll resolve the change requests by myself! Thanks for contributing!

@Rokt33r Rokt33r merged commit 811db3b into BoostIO:master Sep 3, 2020
@Rokt33r Rokt33r added this to the v0.10.0 milestone Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants