Skip to content

Conversation

iqbalcodes6602
Copy link
Contributor

added conditionally rendering for the loaders and remove in other case

fixes: #191

@coveralls
Copy link

coveralls commented Mar 6, 2025

Coverage Status

coverage: 15.776% (-0.7%) from 16.487%
when pulling 4c721b5 on iqbalcodes6602:Get-rid-of-the-full-page-loading
into 4220475 on OneBusAway:main.

Copy link
Member

@aaronbrethorst aaronbrethorst left a comment

Choose a reason for hiding this comment

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

Please run the linter and fix the issues it flags!

@iqbalcodes6602
Copy link
Contributor Author

iqbalcodes6602 commented Mar 6, 2025

Please run the linter and fix the issues it flags!

hi @aaronbrethorst , apologize for the issues. i didn't knew about the linter, have refactored the code as said.
Will do it for other pr also from now.

Copy link
Member

@aaronbrethorst aaronbrethorst left a comment

Choose a reason for hiding this comment

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

looks good 👍

@aaronbrethorst aaronbrethorst merged commit e942d55 into OneBusAway:main Mar 21, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Get rid of the full-page loading indicator on the Arrivals & Departures/Route Schedules page
3 participants