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

Feedback Offsets Page to the Right On Submit #322

Open
darkfrog26 opened this issue Aug 21, 2024 · 7 comments
Open

Feedback Offsets Page to the Right On Submit #322

darkfrog26 opened this issue Aug 21, 2024 · 7 comments
Labels
bug Something isn't working

Comments

@darkfrog26
Copy link

Version

3.1.0

Library

feedback

Flutter channel

beta

Flutter version

3.24.0-0.2.pre

Platform

Web

Details

The feedback opens as expected, and if I close it, everything returns to normal. However, after submitting, the page is shifted to the right:
Screenshot from 2024-08-21 08-53-04

Steps to reproduce

  • Open Feedback
  • Enter text
  • Click Submit

Output of flutter doctor -v

[✓] Flutter (Channel beta, 3.24.0-0.2.pre, on Pop!_OS 22.04 LTS 6.9.3-76060903-generic, locale en_US.UTF-8)
    • Flutter version 3.24.0-0.2.pre on channel beta at /usr/bin/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 7c6b7e9ca4 (3 weeks ago), 2024-07-30 14:26:44 +0700
    • Engine revision 6e4deceb38
    • Dart version 3.5.0 (build 3.5.0-323.2.beta)
    • DevTools version 2.37.2

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/mhicks/Android/Sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /home/mhicks/.local/share/JetBrains/Toolbox/apps/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • Ubuntu clang version 14.0.0-1ubuntu1.1
    • cmake version 3.22.1
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[✓] Android Studio (version 2024.1)
    • Android Studio at /home/mhicks/.local/share/JetBrains/Toolbox/apps/android-studio
    • Flutter plugin version 81.0.2
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0-17.0.11b1207.24-11852314)

[✓] IntelliJ IDEA Ultimate Edition (version 2024.2)
    • IntelliJ at /home/mhicks/.local/share/JetBrains/Toolbox/apps/intellij-idea-ultimate
    • Flutter plugin version 81.1.3
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.92.2)
    • VS Code at /usr/share/code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Pop!_OS 22.04 LTS 6.9.3-76060903-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 127.0.6533.119

[✓] Network resources
    • All expected network resources are available.

• No issues found!
@darkfrog26 darkfrog26 added the bug Something isn't working label Aug 21, 2024
@darkfrog26
Copy link
Author

Three weeks ago and not a single response?

@ueman
Copy link
Owner

ueman commented Sep 10, 2024

This is a hobby project and right now I have other priorities. If this is a huge issue for you, please feel free to submit a bug fix as a PR.

@darkfrog26
Copy link
Author

@ueman, my apologies...I didn't realize this was a single-person project. I'm happy to attempt a fix. Do you have any suggestions of where I might start looking?

@ueman
Copy link
Owner

ueman commented Sep 10, 2024

Unfortunately, I have no idea :(

@CesarArellano
Copy link

CesarArellano commented Sep 30, 2024

@darkfrog26 Hey man, I had the same problem for Web, it seems to be a issue with the screenshot process, that's why we have to put a Future Delayed to wait properly this async process when we call the show method at the moment, something like this:

image

I hope, it will be useful to you buddy

@darkfrog26
Copy link
Author

@CesarArellano you're absolutely right man! Adding the delay fixed the problem. This should probably still be considered a bug, but at least there's a workaround. Thanks!

@CesarArellano
Copy link

CesarArellano commented Sep 30, 2024

@darkfrog26 awesome Matt, it's definitely a bug to be addressed, I hope we get an official fix asap in the package itself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants