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

Basic Popper.js re-implementation of asset tooltips #1009

Merged
merged 3 commits into from
May 28, 2019

Commits on May 23, 2019

  1. Basic Popper.js re-implementation of asset tooltips

    This attempts to display the tooltip to the left of the current asset, but Popper.js is smart enough not to fix it there if that would keep it offscreen. 
    
    There’s just enough CSS to add a border — this could probably use some additional TLC for more distinct colors and we probably want to adjust widths and font sizes, too.
    
    See #998
    acdha committed May 23, 2019
    Configuration menu
    Copy the full SHA
    a07e0cf View commit details
    Browse the repository at this point in the history

Commits on May 24, 2019

  1. Position tooltips below the asset

    This avoids overlapping the thumbnail where possible
    acdha committed May 24, 2019
    Configuration menu
    Copy the full SHA
    7e9955f View commit details
    Browse the repository at this point in the history
  2. Add a hover color to the asset list

    Since the tooltip no longer fills the element with the metadata display, which makes it less obvious which asset is being displayed. This adds a hover outline similar to the active asset color to make those states more distinct.
    acdha committed May 24, 2019
    Configuration menu
    Copy the full SHA
    b41a09e View commit details
    Browse the repository at this point in the history