enviroCar / enviroCar-app

enviroCar Android Application
https://envirocar.org
GNU General Public License v3.0
88 stars 154 forks source link

No actionbar in Settings #818

Open cdhiraj40 opened 2 years ago

cdhiraj40 commented 2 years ago

The back button is missing in setting screen.

Screenshot_2021-09-17-13-33-10-57_47898aacd586656d6f60f8abb18dee77.jpg

cdhiraj40 commented 2 years ago

https://user-images.githubusercontent.com/75211982/133747748-d594ead9-22b5-432e-ad0b-fecff07b049a.mp4

cdhiraj40 commented 2 years ago

@asaikarthikeya I am done with work, I will make PR if you approve. Thanks

asaikarthikeya commented 2 years ago

Sure, go ahead! thanks.

cdhiraj40 commented 2 years ago

Sure, go ahead! thanks.

i will make it tomorrow. my laptop is getting debugged(repaired) :)

SebaDro commented 2 years ago

What's the thing with this issue? Would you please, provide a detailed description for the issue, in accordance to our contributing guidelines

asaikarthikeya commented 2 years ago

What's the thing with this issue? Would you please, provide a detailed description for the issue, in accordance to our contributing guidelines

The back button on the top toolbar is missing in setting screen.

asaikarthikeya commented 2 years ago

@SebaDro Please refer @dewall sir's message in #427, where Dewall sir has mentioned about missing of back option in settings.

SebaDro commented 2 years ago

Yes, @dewall is right. If you have a closer look on the different views, you will notice that the toolbar layout is not consistent across the whole app. There are 3 different layouts:

  1. enviroCar logo with back button
  2. enviroCar logo without back button
  3. text header with back button In addition, there is another interesting design on the help view which switches the header layout using an animation.

Hence, we first should discuss which layout should be preferred for future releases.

Jdbarad commented 2 years ago

I want to contribute to this issue, I also started work on it.

SebaDro commented 2 years ago

Yeah, great. Just go ahead

asaikarthikeya commented 2 years ago

@Jdbarad @SebaDro This issue has been already worked and is yet to be reviewed - PR #821. Please work on a issue that doesnot have a pull request attacked to it. Thanks :)

cdhiraj40 commented 2 years ago

Same

What's the thing with this issue? Would you please, provide a detailed description for the issue, in accordance to our contributing guidelines

The back button on the top toolbar is missing in setting screen.

The same issue is there for StartRecordingActivity, any reason sir? @SebaDro @asaikarthikeya

SebaDro commented 2 years ago

Yes and no :point_down:

Yes, @dewall is right. If you have a closer look on the different views, you will notice that the toolbar layout is not consistent across the whole app. There are 3 different layouts:

  1. enviroCar logo with back button
  2. enviroCar logo without back button
  3. text header with back button In addition, there is another interesting design on the help view which switches the header layout using an animation.

Hence, we first should discuss which layout should be preferred for future releases.

cdhiraj40 commented 2 years ago

Yes and no :point_down:

Yes, @dewall is right. If you have a closer look on the different views, you will notice that the toolbar layout is not consistent across the whole app. There are 3 different layouts:

  1. enviroCar logo with back button
  2. enviroCar logo without back button
  3. text header with back button In addition, there is another interesting design on the help view which switches the header layout using an animation.

Hence, we first should discuss which layout should be preferred for future releases.

yes i know that sir but that activity doesnt have any toolbar

Screenshot_2021-11-11-20-14-57-18_47898aacd586656d6f60f8abb18dee77.jpg

Screenshot_2021-11-11-20-18-16-72_47898aacd586656d6f60f8abb18dee77.jpg

asaikarthikeya commented 2 years ago

Yes and no :point_down:

Yes, @dewall is right. If you have a closer look on the different views, you will notice that the toolbar layout is not consistent across the whole app. There are 3 different layouts:

  1. enviroCar logo with back button
  2. enviroCar logo without back button
  3. text header with back button In addition, there is another interesting design on the help view which switches the header layout using an animation.

Hence, we first should discuss which layout should be preferred for future releases.

yes i know that sir but that activity doesnt have any toolbar

Screenshot_2021-11-11-20-14-57-18_47898aacd586656d6f60f8abb18dee77.jpg

Screenshot_2021-11-11-20-18-16-72_47898aacd586656d6f60f8abb18dee77.jpg

Hello Dhiraj, Do you think a toolbar looks good in the recording screen? Toolbar is intentionally ignored in the recording screen. The back option is also not a must have feature in settings screen. Always try to think in the opposite way(how it will be if I have a toolbar, does it improve the UI?) before asking. Some things are designed intentional keeping several other parameters in mind. Thank you🙂

cdhiraj40 commented 2 years ago

Yes and no point_down

Yes, @dewall is right. If you have a closer look on the different views, you will notice that the toolbar layout is not consistent across the whole app. There are 3 different layouts:

  1. enviroCar logo with back button
  2. enviroCar logo without back button
  3. text header with back button In addition, there is another interesting design on the help view which switches the header layout using an animation.

Hence, we first should discuss which layout should be preferred for future releases.

yes i know that sir but that activity doesnt have any toolbar Screenshot_2021-11-11-20-14-57-18_47898aacd586656d6f60f8abb18dee77.jpg Screenshot_2021-11-11-20-18-16-72_47898aacd586656d6f60f8abb18dee77.jpg

Hello Dhiraj, Do you think a toolbar looks good in the recording screen? Toolbar is intentionally ignored in the recording screen. The back option is also not a must have feature in settings screen. Always try to think in the opposite way( who it will be if I have a toolbar, does it improve the UI?) before asking. Some things are designed intentional keeping several other parameters in mind. Thank youslightly_smiling_face

I meant the back option and not the toolbar in recording screen activity, sorry for the inconvenience. I Will sure try to think that way before asking next time.

pree-T commented 2 years ago

Is this issue still open? May I work on it? Can someone help me?

cdhiraj40 commented 2 years ago

Is this issue still open? May I work on it? Can someone help me?

the issue is not open for work, there is an ongoing discussion for this.

madhurmehta007 commented 1 year ago

@cdhiraj40 Is this issue still in discussion or I can work upon it ?