GauriKashyap / textUtilis

TextUtilis is a react based text editor application.
https://text-utilis-one.vercel.app/
1 stars 9 forks source link

➕ Added Replace text feature #15 #17

Closed SGI-CAPP-AT2 closed 1 year ago

SGI-CAPP-AT2 commented 1 year ago

ShGI

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
text-utilis ✅ Ready (Inspect) Visit Preview Oct 3, 2022 at 4:41AM (UTC)
GauriKashyap commented 1 year ago

@SGI-CAPP-AT2 Good job!....Can you organize the buttons in a better way?

SGI-CAPP-AT2 commented 1 year ago

@SGI-CAPP-AT2 Good job!....Can you organize the buttons in a better way?

yes

SGI-CAPP-AT2 commented 1 year ago

image @GauriKashyap Check this

GauriKashyap commented 1 year ago

How about, having showing the replace word instance, only if the user wants to replace something? Can you give a button in navbar, through which these becomes visible? Also, provide some line spacing between the word replace box and the upper default buttons. Make these changes and send a PR, make sure you don't have any merge conflicts

SGI-CAPP-AT2 commented 1 year ago

How about, having showing the replace word instance, only if the user wants to replace something? Can you give a button in navbar, through which these becomes visible? Also, provide some line spacing between the word replace box and the upper default buttons. Make these changes and send a PR, make sure you don't have any merge conflicts

Okay lemme do it

SGI-CAPP-AT2 commented 1 year ago

@GauriKashyap Check it out and let me know if any issue

GauriKashyap commented 1 year ago

I still feel the indentation of buttons, don't look good, can you make it look better somehow.

Screenshot 2022-10-01 at 4 22 35 PM
SGI-CAPP-AT2 commented 1 year ago

I still feel the indentation of buttons, don't look good, can you make it look better somehow.

Screenshot 2022-10-01 at 4 22 35 PM

You mean margin in between

SGI-CAPP-AT2 commented 1 year ago

image @GauriKashyap check this

SGI-CAPP-AT2 commented 1 year ago

@GauriKashyap Can you look at this again ?

GauriKashyap commented 1 year ago

@GauriKashyap Can you look at this again ?

Yeah make the suitable changes and commit.

SGI-CAPP-AT2 commented 1 year ago

@GauriKashyap Can you look at this again ?

Yeah make the suitable changes and commit.

https://text-utilis-nvu7o20q0-gaurikashyap.vercel.app/ Already done !

SGI-CAPP-AT2 commented 1 year ago

@GauriKashyap Can You look at this again ?

GauriKashyap commented 1 year ago

See the updated website and make the changes accordingly, just make the changes in the updated versions of the website and submit

SGI-CAPP-AT2 commented 1 year ago

@GauriKashyap Let the PR get Merged NOW ! Before any conflicts

SGI-CAPP-AT2 commented 1 year ago

@GauriKashyap

SGI-CAPP-AT2 commented 1 year ago

@GauriKashyap you haven't merged ?