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

RenderFlex overflowed by 6.0 pixels on the bottom #51

Open
rabihmb opened this issue Sep 25, 2020 · 3 comments
Open

RenderFlex overflowed by 6.0 pixels on the bottom #51

rabihmb opened this issue Sep 25, 2020 · 3 comments
Labels
T: Fix Type: :bug: Bug Fixes

Comments

@rabihmb
Copy link

rabihmb commented Sep 25, 2020

Hi, I've installed the package as per the document, it works well but I'm getting a notification in the console as described below:

════════ Exception caught by rendering library ═════════════════════════════════════════════════════
A RenderFlex overflowed by 6.0 pixels on the bottom.
The relevant error-causing widget was: 
  Timetable<BasicEvent>

A screenshots below to help explain my problem

Before
image

so, in order to fix it, I've replaced the following line:

context.timetableTheme?.totalDateIndicatorHeight ?? 72,

with MediaQuery.of(context).size * 0.11 ?? 72

After
image

Environment:

  • Device: Samsung Galaxy Note 5
  • OS version: Android 9.0.0
  • Package version: 0.2.8
@rabihmb rabihmb added the T: Fix Type: :bug: Bug Fixes label Sep 25, 2020
@radikris
Copy link

radikris commented Feb 3, 2021

same here.

@yaroaxito
Copy link

Same issue...

@JonasWanke
Copy link
Owner

Similar to the workaround shown in the issue description, you can configure this height via the theme parameter totalDateIndicatorHeight:

Timetable<BasicEvent>(
  controller: /* … */,
  theme: TimetableThemeData(
    totalDateIndicatorHeight: MediaQuery.of(context).size * 0.11,
    // You might have to use a different value
  ),
),

(not tested)

The issue is fixed in the new pre-releases of v1.0.0 by dynamically adjusting this height to the content instead of using a fixed value.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: Fix Type: :bug: Bug Fixes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants