Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

feat(components) add header tooltip - I120 #151

Conversation

johnnyqbui
Copy link
Contributor

@johnnyqbui johnnyqbui commented Oct 6, 2019

Issue #120

hovering over clause template header title will pop up a tooltip if header has been truncated.

Changes

  • use named imports instead of default from actions to utilize titleGenerator function to display title on tooltip.
  • create state for hover effect.
  • tooltip pops up with full title if header is truncated (> 54 characters in length) and hovered over.
  • Images following, hovering when header title is not truncated:

Screen Shot 2019-10-06 at 12 14 08 AM

  • hovering when header title is truncated:

Screen Shot 2019-10-06 at 11 13 54 AM

Flags

Related Issues

Signed-off-by: J <johnnyqbui7@gmail.com>
@johnnyqbui johnnyqbui force-pushed the johnnyqbui-issue120-header-tooltip-feature branch from 2d45c3e to 24b529d Compare October 6, 2019 05:39
Signed-off-by: J <johnnyqbui7@gmail.com>
@mttrbrts mttrbrts added Difficulty: Medium Hacktoberfest by DigitalOcean and DEV labels Oct 6, 2019
@irmerk irmerk added this to In Progress PR in Hacktoberfest 2019 via automation Oct 6, 2019
@irmerk irmerk added the Type: Feature Request 🛍️ New feature or request label Oct 6, 2019
@irmerk
Copy link
Member

irmerk commented Oct 6, 2019

I'm not able to reproduce this with your branch. I'm not getting the tooltip nor the truncation being in effect. @DianaLease are you able to? The code looks solid, I may be having version issues on my local machine.

Screen Shot 2019-10-06 at 12 25 22 PM

@DianaLease
Copy link
Member

This works great for me both in the cicero-ui demo and when linked with TSv2.

TSv2:
Screen Shot 2019-10-07 at 2 15 12 PM

cicero-ui:
Screen Shot 2019-10-07 at 2 09 09 PM

Nice job!

Hacktoberfest 2019 automation moved this from In Progress PR to Approved PR Oct 7, 2019
Copy link
Member

@DianaLease DianaLease left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@irmerk irmerk merged commit e3cdc3e into accordproject:master Oct 7, 2019
Hacktoberfest 2019 automation moved this from Approved PR to Finished PR Oct 7, 2019
@irmerk
Copy link
Member

irmerk commented Oct 7, 2019

Super cool @johnnyqbui! @Michael-Grover note that this was just merged in.

@Michael-Grover
Copy link
Collaborator

@johnnyqbui very nice, thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Difficulty: Medium Hacktoberfest by DigitalOcean and DEV Type: Feature Request 🛍️ New feature or request
Projects
No open projects
Hacktoberfest 2019
  
Finished PR
Development

Successfully merging this pull request may close these issues.

None yet

5 participants