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: high contrast themes: ListTile selected state is not distinguishable from non-selected state #941

Merged
merged 3 commits into from
Oct 27, 2024

Conversation

Feichtmeier
Copy link
Member

@Feichtmeier Feichtmeier commented Oct 27, 2024

grafik

now the improved high contrast list tiles:

Bildschirmfoto 2024-10-27 um 12 03 30 grafik

Fixes #939

Bonus:

Fixes #940

@Feichtmeier
Copy link
Member Author

okay it works, but we have a new analyzer problem

@HrX03 @Jupi007

   info • The import of 'package:animated_vector/animated_vector.dart' is unnecessary because all of the used elements are also provided by the import of 'package:animated_vector_annotations/animated_vector_annotations.dart' • lib/src/icons/icons_widgets/yaru_animated_icons.dart:1:8 • unnecessary_import
   info • The import of 'package:flutter/widgets.dart' is unnecessary because all of the used elements are also provided by the import of 'package:animated_vector_annotations/animated_vector_annotations.dart' • lib/src/icons/icons_widgets/yaru_animated_icons.dart:3:8 • unnecessary_import

when I remove it yaru_animated_icons.dart is missing the import O.o
whats happening?

@HrX03
Copy link
Contributor

HrX03 commented Oct 27, 2024

okay it works, but we have a new analyzer problem

@HrX03 @Jupi007

   info • The import of 'package:animated_vector/animated_vector.dart' is unnecessary because all of the used elements are also provided by the import of 'package:animated_vector_annotations/animated_vector_annotations.dart' • lib/src/icons/icons_widgets/yaru_animated_icons.dart:1:8 • unnecessary_import
   info • The import of 'package:flutter/widgets.dart' is unnecessary because all of the used elements are also provided by the import of 'package:animated_vector_annotations/animated_vector_annotations.dart' • lib/src/icons/icons_widgets/yaru_animated_icons.dart:3:8 • unnecessary_import

when I remove it yaru_animated_icons.dart is missing the import O.o
whats happening?

I moved all of the required imports for generated files into the animated_vector_annotations file, I think the yaru lib will need to update to not include other stuff inside those files

@Jupi007
Copy link
Member

Jupi007 commented Oct 27, 2024

Weird, I can't reproduce the analyzing error locally using your branch 🤔
Okay, I reproduced it

@Jupi007
Copy link
Member

Jupi007 commented Oct 27, 2024

Yeah, it looks like you just need to drop animated_vector.dart and widgets.dart:

- import 'package:animated_vector/animated_vector.dart';
import 'package:animated_vector_annotations/animated_vector_annotations.dart';
- import 'package:flutter/widgets.dart';

@Feichtmeier
Copy link
Member Author

Yeah, it looks like you just need to drop animated_vector.dart and widgets.dart:

- import 'package:animated_vector/animated_vector.dart';
import 'package:animated_vector_annotations/animated_vector_annotations.dart';
- import 'package:flutter/widgets.dart';

no I dont think this works : D

now I got 777 x Undefined class 'AnimatedVectorData'.
Try changing the name to the name of an existing class, or creating a class with the name 'AnimatedVectorData'.

@HrX03
Copy link
Contributor

HrX03 commented Oct 27, 2024

Yeah, it looks like you just need to drop animated_vector.dart and widgets.dart:

- import 'package:animated_vector/animated_vector.dart';
import 'package:animated_vector_annotations/animated_vector_annotations.dart';
- import 'package:flutter/widgets.dart';

no I dont think this works : D

now I got 777 x Undefined class 'AnimatedVectorData'.
Try changing the name to the name of an existing class, or creating a class with the name 'AnimatedVectorData'.

I have just checked and the member is exported from the file, did you update every animated_vector dep and ran pub get/upgrade? Also try restarting the analyzer, I'm sure the member is exported

@Feichtmeier
Copy link
Member Author

Yeah, it looks like you just need to drop animated_vector.dart and widgets.dart:

- import 'package:animated_vector/animated_vector.dart';
import 'package:animated_vector_annotations/animated_vector_annotations.dart';
- import 'package:flutter/widgets.dart';

no I dont think this works : D
now I got 777 x Undefined class 'AnimatedVectorData'.
Try changing the name to the name of an existing class, or creating a class with the name 'AnimatedVectorData'.

I have just checked and the member is exported from the file, did you update every animated_vector dep and ran pub get/upgrade? Also try restarting the analyzer, I'm sure the member is exported

jeeez, sorry.

Copy link
Member

@Jupi007 Jupi007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@Feichtmeier
Copy link
Member Author

the goldens test passes locally, we need the nightly for this I think so this can be ignored

@Feichtmeier Feichtmeier merged commit ebddaed into main Oct 27, 2024
6 of 7 checks passed
@Feichtmeier Feichtmeier deleted the Feichtmeier/issue939 branch October 27, 2024 11:45
This was referenced Oct 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants