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

Change icon for the layer tools in the layer manager panel on the map page #8376

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

Conversation

MichelGabriel
Copy link
Contributor

@MichelGabriel MichelGabriel commented Sep 17, 2024

The tools in the layer manager in the map are opened on hover. But the icon was misleading, it invited you to click on it. When you hovered the layer the icon disappeared leading to the thought that you couldn't click the icon and therefore couldn't open a menu and missing some extra options (that weren't there).

The tools are still opened on hover, making it clickable asks for some rebuilding, but the icon has changed and the color is dimmed.

Related to issue: #8116

The new look:

gn-map-layer-panel

Checklist

  • I have read the contribution guidelines
  • Pull request provided for main branch, backports managed with label
  • Good housekeeping of code, cleaning up comments, tests, and documentation
  • Clean commit history broken into understandable chucks, avoiding big commits with hundreds of files, cautious of reformatting and whitespace changes
  • Clean commit messages, longer verbose messages are encouraged
  • API Changes are identified in commit messages
  • Testing provided for features or enhancements using automatic tests
  • User documentation provided for new features or enhancements in manual
  • Build documentation provided for development instructions in README.md files
  • Library management using pom.xml dependency management. Update build documentation with intended library use and library tutorials or documentation

… page.

The tools in the layer manager in the map are opened on hover. But the icon was misleading, it invited you to click on it. When you hovered the layer the icon disappeared leading to the thought that you couldn't click the icon and therefore couldn't open a menu and missing some extra options (that weren't there).

The tools are still opened on hover, making it clickable asks for some rebuilding, but the icon has changed and the color is dimmed.
@wangf1122
Copy link
Collaborator

wangf1122 commented Sep 17, 2024

@MichelGabriel

I have cherry pick the commit to our environment (4.2.9) with HNAP schema. Deployed and got the looking like this.

image

Somehow there is some overlapping with layer removal buttons and this newly added arrow icon of the page. It makes the icon not clickable.

I will do some confirmation test on the main branch to the latest GN.

@josegar74 josegar74 added this to the 4.4.6 milestone Sep 18, 2024
@wangf1122
Copy link
Collaborator

@MichelGabriel

Here is the result I tested from latest GN4. If I dont hover my mouse, it looks ok.
image

But if I try to hover my mouse to the arrow, the three pop buttons shows and there is no way I can click on that arrow sign.

image

Maybe we should offset either one a little? I dont know if screen resolution matter. But here is what my Windows OS setup

image

@MichelGabriel
Copy link
Contributor Author

The tools menu is now opened on click

gn-layermanager

@wangf1122
Copy link
Collaborator

@MichelGabriel

It looks good.

image

@ianwallen ianwallen modified the milestones: 4.4.6, 4.4.7 Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants