mirror of
https://github.com/flutter/samples.git
synced 2026-06-25 23:58:44 +00:00
5f4c795695
## Description **Problem:** `LogoutViewModel` was built inside `HomeHeader.build()`, so each rebuild could create a new instance and a new `logout` command. That does not match how other Compass view models are scoped (created in the GoRouter route `builder` and passed in), and it can affect an in-flight logout when the home screen rebuilds. **Change:** Create `LogoutViewModel` once in the `/home` route next to `HomeViewModel`, pass it through `HomeScreen` → `HomeHeader` → `LogoutButton`, and remove inline construction in `home_title.dart`. Update `home_screen_test.dart` to build `LogoutViewModel` with fakes and remove `Provider` wrappers that only supported `context.read()` in the header. **Result:** Logout view model lifetime aligns with the home route; no visual or copy changes (screenshots not needed). Fixes https://github.com/flutter/samples/issues/2604 --- ## Pre-launch Checklist - [x] I read the [Flutter Style Guide] _recently_, and have followed its advice. - [x] I signed the [CLA]. - [x] I read the [Contributors Guide]. - [x] I have added sample code updates to the [changelog]. - [x] I updated/added relevant documentation (doc comments with `///`). If you need help, consider asking for advice on the #hackers-devrel channel on [Discord]. <!-- Links --> [Flutter Style Guide]: https://github.com/flutter/flutter/blob/master/docs/contributing/Style-guide-for-Flutter-repo.md [CLA]: https://cla.developers.google.com/ [Discord]: https://github.com/flutter/flutter/blob/master/docs/contributing/Chat.md [Contributors Guide]: https://github.com/flutter/samples/blob/main/CONTRIBUTING.md [changelog]: ../CHANGELOG.md --------- Co-authored-by: Eric Windmill <eric@ericwindmill.com>
77 lines
2.2 KiB
Dart
77 lines
2.2 KiB
Dart
// Copyright 2024 The Flutter team. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
import 'package:flutter/material.dart';
|
|
import 'package:google_fonts/google_fonts.dart';
|
|
|
|
import '../../auth/logout/view_models/logout_viewmodel.dart';
|
|
import '../../auth/logout/widgets/logout_button.dart';
|
|
import '../../core/localization/applocalization.dart';
|
|
import '../../core/themes/dimens.dart';
|
|
import '../view_models/home_viewmodel.dart';
|
|
|
|
class HomeHeader extends StatelessWidget {
|
|
const HomeHeader({
|
|
super.key,
|
|
required this.viewModel,
|
|
required this.logoutViewModel,
|
|
});
|
|
|
|
final HomeViewModel viewModel;
|
|
final LogoutViewModel logoutViewModel;
|
|
|
|
@override
|
|
Widget build(BuildContext context) {
|
|
final user = viewModel.user;
|
|
if (user == null) {
|
|
return const SizedBox();
|
|
}
|
|
return Column(
|
|
crossAxisAlignment: CrossAxisAlignment.stretch,
|
|
children: [
|
|
Row(
|
|
mainAxisAlignment: MainAxisAlignment.spaceBetween,
|
|
crossAxisAlignment: CrossAxisAlignment.start,
|
|
children: [
|
|
ClipOval(
|
|
child: Image.asset(
|
|
user.picture,
|
|
width: Dimens.of(context).profilePictureSize,
|
|
height: Dimens.of(context).profilePictureSize,
|
|
),
|
|
),
|
|
LogoutButton(viewModel: logoutViewModel),
|
|
],
|
|
),
|
|
const SizedBox(height: Dimens.paddingVertical),
|
|
_Title(text: AppLocalization.of(context).nameTrips(user.name)),
|
|
],
|
|
);
|
|
}
|
|
}
|
|
|
|
class _Title extends StatelessWidget {
|
|
const _Title({required this.text});
|
|
|
|
final String text;
|
|
|
|
@override
|
|
Widget build(BuildContext context) {
|
|
return ShaderMask(
|
|
blendMode: BlendMode.srcIn,
|
|
shaderCallback: (bounds) => RadialGradient(
|
|
center: Alignment.bottomLeft,
|
|
radius: 2,
|
|
colors: [Colors.purple.shade700, Colors.purple.shade400],
|
|
).createShader(Rect.fromLTWH(0, 0, bounds.width, bounds.height)),
|
|
child: Text(
|
|
text,
|
|
style: GoogleFonts.rubik(
|
|
textStyle: Theme.of(context).textTheme.headlineLarge,
|
|
),
|
|
),
|
|
);
|
|
}
|
|
}
|