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

Mouse over an image to see it in its original size #6

Open
banool opened this issue Sep 9, 2018 · 5 comments
Open

Mouse over an image to see it in its original size #6

banool opened this issue Sep 9, 2018 · 5 comments
Labels
enhancement New feature or request

Comments

@banool
Copy link
Owner

banool commented Sep 9, 2018

The images can be a bit small, this would help with visibility. Most images will be larger than 130px square so in general this would be larger. You'd have to set a cap on how big you blow it up to.

Consider whether it should go back to normal when the cursor moves beyond the edge of the original image or the blown up image. I think the original.

@banool banool added the enhancement New feature or request label Sep 9, 2018
banool added a commit that referenced this issue Sep 13, 2018
@banool
Copy link
Owner Author

banool commented Sep 13, 2018

I address this partially in 816d726. The hover works (including any highlighting and other overlays), but there there are two issues:

  1. It doesn't scale to its original size. I think it's really better this way, but it could be worth experimenting with scaling to original size up to a cap.
  2. The hover doesn't go away until you leave the bounds of the newly scaled up image. This is annoying, and would be very annoying if we implemented 1. I tried to get it that the hover goes away if you move beyond the bounds of the original image size, but I couldn't get it to work.

I'm leaving this issue open for now in case we decide to improve it further, but it works pretty well for now.

@banool
Copy link
Owner Author

banool commented Sep 13, 2018

For this and any other setting, there should be a way to turn it off. The best way to do this will be to make the hover variant a class of its own and if the setting is disabled, use the non-hover variant.

@banool
Copy link
Owner Author

banool commented Sep 13, 2018

Turns out this doesn't work on all browsers. On chrome this displays weirdly, the z-index isn't doing what it should.

@banool
Copy link
Owner Author

banool commented Sep 22, 2018

Also before I would consider this done, there needs to be an option in the settings to disable this behaviour.

@banool
Copy link
Owner Author

banool commented Sep 22, 2018

As of 4d8b850 this is mostly done. All of the above issues have been resolved except:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant