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

Update HyperLinkCell.cs #318

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update HyperLinkCell.cs #318

wants to merge 2 commits into from

Commits on Mar 20, 2020

  1. Update HyperLinkCell.cs

    I've made all the modifications based on the behavior of the Latest Excel under Windows: the cursor change only when the mouse is over the text.
    
    To achieve that:
    1. I've changed the name of the property from IsPressed to IsOverLink.
    2. I've removed OnStartEdit because you can still edit the cell even if the mouse is over the link.
    3. I've remove the event OnMouseEnter and replaced them with OnMouseMove. OnMouseLeave is still there.
    4. OnMouse mouse check if the mouse is over the text and change IsOverLink accordingly and manage the cursor
    5. For each events, the default behavior is kept. Example: Even if the mouse is down over the text, you can still move outside the cell and select a range
    6. In Excel, with the mouse down over the link, if you move the mouse before mouse up, the navigate process is stopped. Unfortunately, I couldn't reproduce since when the mouse is down, OnMouseMove isn't receiving any event. Not sure why. Same for OnMouseLeave, not sure it's useful.
    lamazze authored Mar 20, 2020
    Configuration menu
    Copy the full SHA
    e3bcd7b View commit details
    Browse the repository at this point in the history

Commits on Mar 22, 2020

  1. some fixes

    jingwood committed Mar 22, 2020
    Configuration menu
    Copy the full SHA
    d75d96b View commit details
    Browse the repository at this point in the history