-
Notifications
You must be signed in to change notification settings - Fork 29
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 Outbound Links to Relevant Sites #1187
Comments
Notes from email: The functionality is great. I think we should consider removing the Cyclomedia button at the bottom of the page as the one here effectively replaces it. I think we need to give each icon a bit more room to breathe by extending the spacing horizontally to fill more of the width of the window. I also think we should treat them as full buttons instead of just icons, which will give the on-hover state more prominence. Adding extra space between the buttons will also make the tooltips easier to read as the mouse will have to travel farther before one tooltip drops out and the next pops up. Speaking on these, can we adjust them so that they disappear on the instant the mouse stops hovering, rather than a few moments later? This is causing a situation where one tooltip will overlap the adjacent one for a fraction of a second which is disorienting. I will aim to provide some (simple) design mockups for these by tomorrow. I realize extending the horizontal space might clash with the other feature to compare two Tax Lots, but I have included some thoughts below on addressing that. |
@jpiacentinidcp The Cyclomedia button has been removed, along with the I'm not sure what you mean by "we should treat them as full buttons instead of just icons, which will give the on-hover state more prominence", can you please clarify? I've added spacing between the buttons (and they will wrap to a second row for multi-lot selections), please let me know if this matches what you were thinking. I've also removed the delay for the tooltip disappearing. This does make it difficult to click the links in the tooltips. We are able to set the delay for each tooltip individually, so we could re-insert the delay for those tooltips. There are three with links, so we could also switch the order of the icons to make it so ones with links do not appear consecutively. Let me know what you'd like to do. |
Thanks @dhochbaum-dcp ! I originally envisioned the icons having boxes around them that would adjust in color on hover to more strongly communicate to users they are clickable buttons, but with the extra spacing I think that is perhaps unnecessary for now. I agree with returning the delay (perhaps 0.25 seconds) to the PFF, CPE, and EDDE tooltips only so a user can easily mouse to the links. Let's rearrange the order to as follows: ZAP (We'll need to confirm the radius filter is fixed) |
@jpiacentinidcp Sounds good, I've pushed those updates. |
@dhochbaum-dcp Thanks, I think this works great! |
Add a set of icons to the pages of tax lots which link to different sites, to pages targeting that specific location (where possible).
The sites, in order of the icons, are:
The text was updated successfully, but these errors were encountered: