responsive_utils.dart not included as an export in responsive_framework.dart

[mstainer14]

When developing for web, the breakpointsLandscape property of ResponsiveBreakpoints.builder does not work as intended because the default value of _landscapePlatforms in responsive_breakpoints.dart is set to the following:

  static const List<ResponsiveTargetPlatform> _landscapePlatforms = [
    ResponsiveTargetPlatform.iOS,
    ResponsiveTargetPlatform.android,
    ResponsiveTargetPlatform.fuchsia,
  ];

In order to overcome this we need to set the landscapePlatforms property of ResponsiveBreakpoints.builder. However, because responsive_utils.dart is not included as an export in the barrel file, we cannot access the ResponsiveTargetPlatform class directly but have to manually import the file import 'package:responsive_framework/src/utils/responsive_utils.dart'; which gives us the following error.

Import of a library in the 'lib/src' directory of another package.
Try importing a public library that exports this library, or removing the import.dart[implementation_imports](https://dart.dev/lints/implementation_imports)

library package:responsive_framework/src/utils/responsive_utils.dart

Is it possible to add the responsive_utils.dart to the export file or move to a new location that is accessible through the import?

Many thanks

[mstainer14]

@rayliverified

[rayliverified]

Let's add it ASAP. That's a serious issue. Can you open a PR and I'll add you as a contributor too.

…

On Fri, 21 Jun 2024, 12:18 am Mike Stainer, ***@***.***> wrote: @rayliverified <https://github.com/rayliverified> — Reply to this email directly, view it on GitHub <#181 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEXZO3QB6W7E4BVMHEOQ4KTZIOZQXAVCNFSM6AAAAABJVFYBVWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBSGAYTONBTGU> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[mstainer14]

#182

[Juliotati]

@rayliverified Any prevision on when this gets published?
I noticed there is no version bump on the PR 👀 (and it’s not linked to this issue either)

[rayliverified]

Not yet. I need to bundle some other changes for publishing.