riverscuomo / public-bug-hunt

A place to file bug reports for all of my apps
GNU General Public License v3.0
6 stars 0 forks source link

Small Appbar missing Circle Avatar, good evening, and create playlists button #41

Closed KCreek1 closed 11 months ago

KCreek1 commented 11 months ago

Describe the bug There is no way to get to the profile screen. Prohibits signing out/switching accounts. Or if a lurker - being able to make account.

To Reproduce Steps to reproduce the behavior:

  1. Open app
  2. Only market, search, and your library are available.
  3. No option for profile.
  4. Tried clicking on pfp in market but nothing happened.

Expected behavior Should have access to profile page.

Screenshots https://github.com/riverscuomo/public-bug-hunt/assets/144278926/a4dafe83-7b7b-45ef-a54f-e763026e1ee9

Desktop (please complete the following information): N/A

Smartphone (please complete the following information):

Additional context None

riverscuomo commented 11 months ago

can you tap on your avatar top right?

riverscuomo commented 11 months ago

image

KCreek1 commented 11 months ago

The only thing I show top right is the search. Is it supposed to be to the right of that? I don't see it on small screen.

image

SlightlyCompletely commented 11 months ago

Also no avatar on iPhone 7 (running iOS 15.7.9) in Weezify v1.9.0 (170). (It does show on iPad Air though.)

KCreek1 commented 11 months ago

I tried rotating my phone. Then it formats like an iPad and I can see the profile. If I click on it then rotate back to portrait view - my profile is there.

https://github.com/riverscuomo/public-bug-hunt/assets/144278926/62304a3d-c488-4a98-8fa8-2c515235675a

SlightlyCompletely commented 11 months ago

Even in landscape orientation, my iPhone fails to show the avatar.

Device: iPhone 7 OS: iOS 15.7.9 Weezify v1.9.0

https://github.com/riverscuomo/public-bug-hunt/assets/144279871/314ba44a-df97-4386-bfe4-f943fb35c681

riverscuomo commented 11 months ago

problem is here either no user or currentroute doesn't contain "/"


/// The appbar for the small screen on both Weezify and Rivify
class SmallAppBar extends StatelessWidget {
  const SmallAppBar({
    super.key,
    required this.profileImageUrl,
    this.toolTipText = 'Search your library',
    required this.isAnonymous,
    required this.playerBloc,
    this.user,
    required this.currentRoute,
  });

  final String profileImageUrl;
  final String toolTipText;
  final bool isAnonymous;
  final BasePlayerBloc playerBloc;
  final User? user;
  final String currentRoute;

  @override
  Widget build(BuildContext context) {
    final _onHomeScreen = currentRoute.contains('"/"');
    final _onPlayerSearchScreen = currentRoute.contains('playerSearch');
    return AppBar(
      backgroundColor: const Color.fromRGBO(18, 18, 18, 1),
      title: Row(
        children: [
          if (user != null && _onHomeScreen)
KCreek1 commented 11 months ago

All show at the top.

iPhone SE 2nd gen iOS 16.6.1 Weezify 1.9.0 (171)

Safari browser Web app (small screen) 1.9.0 (171)

image