AOSSIE-Org / AOSSIE-Org/NeuroTrack

BUG: Discarded Provider.of call in AuthScreen causes unnecessary widget rebuilds

Open
#186 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Dart
Stars
29
Forks
50
PR merge metrics
No merged PRs in 30d

Description

### Is there an existing issue for this?

- [x] I have searched the existing issues

### What happened?

## Description
In `auth_screen.dart`, inside the `build()` method, there is a `Provider.of`
call whose return value is immediately discarded:

## Current Behavior
Provider.of(context, listen: true).authNavigationStatus;

This line forces a full widget rebuild every time AuthProvider changes,
even though the value is never used in the UI. This is a silent performance
issue that can cause janky behavior on low-end devices.

## Expected Behavior
Rebuilds should only happen when the value is actually consumed by the UI.
The navigation logic is already handled in `didChangeDependencies`,
making this line redundant.

## Proposed Fix
Remove the redundant line entirely, since `didChangeDependencies` already
reacts to AuthProvider changes for navigation purposes.

## File
lib/presentation/auth/auth_screen.dart — build() method

### Record

- [x] I agree to follow this project's Code of Conduct
- [x] I want to work on this issue

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.