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

fix issue with position offset when the always show tooltip enabled is true #145

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NicosNicolaou16
Copy link

@NicosNicolaou16 NicosNicolaou16 commented Apr 19, 2024

Fixed an issue with tool tip position offset when the alwaysShowTooltip is true and set direction with FlutterSliderTooltipDirection.top (optional), so the tool tip is not showing or the handler overlap the tool tip

The fix is in this file: lib/src/view/flutter_slider_widget.dart

You can try the "master" branch with this code to see the issue and also you can see the issue below from print screens

Container(
         // width: 100,
         // height: 50,
         padding: EdgeInsets.all(50),
             child: FlutterSlider(
                values: [0],
                max: 100,
                min: 0,
                tooltip: FlutterSliderTooltip(
                    alwaysShowTooltip: true,
                    direction: FlutterSliderTooltipDirection.top,
                    //positionOffset: FlutterSliderTooltipPositionOffset(top: -25) //added this line to fix it temporary 
              ),
           ),
        );

Screenshot_20240419_232057
Screenshot_20240419_232126
Screenshot_20240419_232137

@NicosNicolaou16 NicosNicolaou16 changed the title fix issue with position offset when the always show tooltip enabled (true) fix issue with position offset when the always show tooltip enabled is true Apr 22, 2024
@NicosNicolaou16
Copy link
Author

Hello 👋 can you review my pull request please? 😁🙏@Ali-Azmoud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant