allmaps / editor

https://editor.allmaps.org/
7 stars 2 forks source link

Icon visual issues #40

Open coret opened 1 year ago

coret commented 1 year ago

image

  1. some icons are transparent on page load (see for example link) although images do have mask/gcp's, when image is selected icons do lit up (with other batch this issue isn't present).
  2. the mask icon is always shown if there there are gcp's even when no mask is drawn, although technically correct (when no mask is drawn the whole image is the mask) this makes it hard to see in the overview which images have mask drawn (and which not).
  3. the icons both contain alt values, respectively "Image contains georeferenced map" and "Image contains masked map", but this doesn't provide a tooltip when hovering over an icon. Suggestion: add a title to both icons (can have same value as alt) so users more easily understand the meaning.
  4. the icons shown with the images differ from the Georeference and Mask icons used in the top op the page, I think this is confusing.
  5. the icons are in "reverse" order compared to the top of the page.
  6. the icons on the image in the bottom right position are hidden by the "drawer" (see image above). Suggestion: change bottom: 5px in the .icons class to top: 5px
bertspaan commented 1 year ago

Very good comments, thanks! I'll probably fix a few of these issues soon, but we're also thinking about a bigger overhaul of Allmaps Editor - that will probably not happen until later this year or early next year (we need to find funding first!)