search

recyclinginnova / recyling-app

recycling app
0 stars 0 forks source link

Material info #12

Closed tasnmb closed 2 years ago

tasnmb commented 2 years ago

Added navbar

tasnmb commented 2 years ago

Hey,

I'm fine with creating separate files for the screens and will commit that shortly,

However I'm confused by the request to wrap the Navigation Bar in its own widget. The navbar I used is part of the Scaffold class. If I move the navbar to its own widget and run, then the app bar will not show up.

as shown by here: image

Therefore I will not make that change for now.

Also, the variable name index was chosen as it stores a numerical value, which is the index of the current screen needed in the screens list. It is also based off the selectedIndex variable in the NavigationBar class.

I am happy to change it, however I feel that index or currentIndex is a more suitable name.

tasnmb commented 2 years ago

Please