project_threejs_ai icon indicating copy to clipboard operation
project_threejs_ai copied to clipboard

'toggle'

Open blaqbrane opened this issue 2 years ago • 8 comments

I added a new feature that toggles the editor tab contents(filepicker, colorpicker and aipicker) on and off on click. I discovered for example when the colorpicker is selected on mobile device after a different color is picked, the colorpicker still covers part of the model thereby preventing full visualization. Thanks for going through my little contribution.

blaqbrane avatar Apr 30 '23 23:04 blaqbrane

how did you handle having 2 onclick events?

Raspber avatar May 04 '23 22:05 Raspber

I used arrow function to handle the onClick event. It toggles the value of the isopen state variable and sets the activeEditorTab state variable based on the current value of isopen, it's just one onclick event.

On Thu, May 4, 2023, 23:11 Hauvert @.***> wrote:

how did you handle having 2 onclick events?

— Reply to this email directly, view it on GitHub https://github.com/adrianhajdin/project_threejs_ai/pull/10#issuecomment-1535475694, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZDLBTKR5NDTZ25OTD4HRFDXEQSRRANCNFSM6AAAAAAXRHS44E . You are receiving this because you authored the thread.Message ID: @.***>

blaqbrane avatar May 04 '23 22:05 blaqbrane

i like it. can we change the statements so if a tab is open and another tab is selected instead of closing the tab it should just switch the selected tab. then if you select the same tab it will close.

Raspber avatar May 06 '23 19:05 Raspber

Sure, I will do just that and let you know

On Sat, May 6, 2023, 20:22 Hauvert @.***> wrote:

i like it. can we change the statements so if a tab is open and another tab is selected instead of closing the tab it should just switch the selected tab. then if you select the same tab it will close.

— Reply to this email directly, view it on GitHub https://github.com/adrianhajdin/project_threejs_ai/pull/10#issuecomment-1537206750, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZDLBTJ5FW2MECQP3KMBMDTXE2QF3ANCNFSM6AAAAAAXRHS44E . You are receiving this because you authored the thread.Message ID: @.***>

blaqbrane avatar May 09 '23 16:05 blaqbrane

I have made the changes such that if a tab is open and a user selects another tab it switches to the new one and if the same tab is selected it closes the tab.

On Tue, May 9, 2023, 17:46 NNAMANI EBUBE @.***> wrote:

Sure, I will do just that and let you know

On Sat, May 6, 2023, 20:22 Hauvert @.***> wrote:

i like it. can we change the statements so if a tab is open and another tab is selected instead of closing the tab it should just switch the selected tab. then if you select the same tab it will close.

— Reply to this email directly, view it on GitHub https://github.com/adrianhajdin/project_threejs_ai/pull/10#issuecomment-1537206750, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZDLBTJ5FW2MECQP3KMBMDTXE2QF3ANCNFSM6AAAAAAXRHS44E . You are receiving this because you authored the thread.Message ID: @.***>

blaqbrane avatar May 12 '23 13:05 blaqbrane

thanks! im adding a product selector component to it. when i get it figured out all the way ill upload that as well.

Raspber avatar May 16 '23 01:05 Raspber

Thanks for improvements

chauhanarya007 avatar May 20 '23 19:05 chauhanarya007

Hi Adrian I hope this finds you well, I created a web version of a job site using jsearch Api and React, would love your feedback thanks Below is the link to the site.

https://remotej.netlify.app

On Tue, May 9, 2023, 17:46 NNAMANI EBUBE @.***> wrote:

Sure, I will do just that and let you know

On Sat, May 6, 2023, 20:22 Hauvert @.***> wrote:

i like it. can we change the statements so if a tab is open and another tab is selected instead of closing the tab it should just switch the selected tab. then if you select the same tab it will close.

— Reply to this email directly, view it on GitHub https://github.com/adrianhajdin/project_threejs_ai/pull/10#issuecomment-1537206750, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZDLBTJ5FW2MECQP3KMBMDTXE2QF3ANCNFSM6AAAAAAXRHS44E . You are receiving this because you authored the thread.Message ID: @.***>

blaqbrane avatar Jun 22 '23 11:06 blaqbrane