smaranjitghose / DocLense

An open-source document scanner!
https://doclense.vercel.app/
Creative Commons Zero v1.0 Universal
147 stars 131 forks source link

Reducing the number of screen #266

Closed anushbhatia closed 2 years ago

anushbhatia commented 3 years ago

In the current Model

  1. Two navigation screens are present one for cropping and one for filters. Instead, we can minimize it to one. WhatsApp Image 2021-05-21 at 12 40 45 AM (2) WhatsApp Image 2021-05-21 at 12 40 45 AM (3)

Having both options on the same screen reduce the navigation part of it.

rudranshsharma123 commented 3 years ago

Hello, I am a participant of LGM-SOC. I am assuming all the widgets and logic from the Filters need to be transported into the main screen after taking the picture? I would love to work on this issue.

anushbhatia commented 3 years ago

Hello, I am a participant of LGM-SOC. I am assuming all the widgets and logic from the Filters need to be transported into the main screen after taking the picture? I would love to work on this issue.

Yeah, the filter's button is present there you just have to move this here. If you face a problem with undo and redo function you can move it up.

ayyysh04 commented 2 years ago

Already working on this task. Please assign this task to me.

ayyysh04 commented 2 years ago

Done the task and created the pull request ,please review the same @anushbhatia

ayyysh04 commented 2 years ago

Done the task and created the pull request ,please review the same @anushbhatia

@smaranjitghose

anushbhatia commented 2 years ago

Sure will review it and let you know

ayyysh04 commented 2 years ago

@anushbhatia you can close this issue now.😀