~~Scrollbar Zoom Buttons Obscured~~ --> Remove buttons

Issue #467 resolved
Robert Leach created an issue

USE CASE: WHAT DO YOU WANT TO DO?

Zoom an edge using the scrollbar-associated zoom buttons

STEPS TO REPRODUCE AN ISSUE (OR TRIGGER A NEW FEATURE)

  1. Open small_133x133.txt
  2. Try to click the +/- zoom buttons associated with any of the IMV's scrollbars

CURRENT BEHAVIOR

It appears that a recent update messed up the layout of the zoom buttons associated with the scrollbars. You can only see a small portion of 1 (or 2?) of the +/- buttons at both ends of both IMV scrollbars:

scrollbar-zoombtns-obscured.png

EXPECTED BEHAVIOR

The fill +/- buttons should be visible and clickable.

DEVELOPERS ONLY SECTION

SUGGESTED CHANGE (Pseudocode optional)

Looks like the scrollbar size needs to shrink a little to accommodate the buttons.

FILES AFFECTED (where the changes will be implemented) - developers only

unknown

LEVEL OF EFFORT - developers only

minor

COMMENTS

Comments (10)

  1. Christopher Keil repo owner

    @hepcat72 I noticed this only after I switched to the Mac for development... there have been button issues with OSX before. They seem to not shrink beyond a certain size due to some minimum font size of the + - signs.

    A solution might be to make the font size OS dependent. Maybe the check for the OS can be moved to the Helper utility class and be statically called instead of residing in TreeView3.java (I think).

  2. Christopher Keil repo owner

    This issue has been edited so that the old 1-tile zoom buttons should be removed.

  3. Christopher Keil repo owner

    Removal of incremental scale buttons

    This issue has been changed. Incremental scale buttons have been removed in favor of a more intuitive interface and to reduce UI clutter. All associated code (layout, listeners) has been cleared as well. Tested on Win10, OSX to follow

    Resolves: #467

    → <<cset 1a4ef2eadb6b>>

  4. Robert Leach reporter

    BTW, they're more than just 1 tile zoom buttons (for 1-side/1-dimension zooming). Modifier keys make them zoom more than a tile, to differing degrees based on the modifier.

  5. Christopher Keil repo owner

    Removal of incremental scale buttons

    This issue has been changed. Incremental scale buttons have been removed in favor of a more intuitive interface and to reduce UI clutter. All associated code (layout, listeners) has been cleared as well. Tested on Win10, OSX to follow

    Resolves: #467

    → <<cset 1a4ef2eadb6b>>

  6. Log in to comment