FlutterFlow / flutterflow-issues

A community issue tracker for FlutterFlow.
120 stars 18 forks source link

Reorder list not useable on dark themes #2900

Open Aging-Developer opened 4 months ago

Aging-Developer commented 4 months ago

Has your issue been reported?

Current Behavior

there is no way to change the color of the element that is being dragged in a reorderable list. It seems to be a white box with no custom options.

So when you have a list with a theme (dark background and white text), the box is white and the text is also white, so it looks blank, meaning the the reorderable lists are totally useless if you have dark themes with white text.

Expected Behavior

to override the dragged items colors so it is visible, if the theme is using white as the item text.

When the reorderable flag is set, the reorderableList widget should be wrapped with:

Theme( data: Theme.of(context).copyWith( canvasColor: Colors.transparent, shadowColor: Colors.transparent, ), child: ReorderableListView( ... ), )

Steps to Reproduce

  1. create a page with a dark BG
  2. add a list with reorderable enabled
  3. create a list item where the text is white.

Run it, the dragged item looks blank, as it is white on white.

Reproducible from Blank

Bug Report Code (Required)

na

Context

cannot use reordable lists with dark themes.

Visual documentation

n/a.

Additional Info

n/a

Environment

- FlutterFlow version: 4.1.49+
- Platform:
- Browser name and version:
- Operating system and version affected:

General

Relative to the time the changes were made, data was lost within

When following my steps to reproduce, data loss happens

ignalauret commented 4 months ago

Hey @Aging-Developer thanks for your report. I can confirm that there is no easy way to change the color of the reorder icon, so if you want to put a White list item, you will not see the reorder icon. I will send this to our technical team so they can check if we can add some easy way of editing it. Have a great day!

Aging-Developer commented 4 months ago

Hi, thank you...... just to point out its not an icon, its the row text when the element is dragged.

it can be achieved by the technical team adding a parameter to the designer, which would generate this code:

Theme( data: Theme.of(context).copyWith( canvasColor: Colors.transparent, shadowColor: Colors.transparent, ), child: ReorderableListView( ... ), )

Aging-Developer commented 3 months ago

@leighajarett Can someone confirm when this is likely to be done?

  1. It doesn't work well without modifying the code
  2. It causes so much wasted time having to remodify the code every time FF does an update the regens the dart code.

Its an easy fix for you guys to do, I have even include the code that needs to be generated, although a good implementation would be to have these as properties of a reorder list in the FF designer.