chatGPTBox icon indicating copy to clipboard operation
chatGPTBox copied to clipboard

UI fixes & improvments

Open mhd-fettah opened this issue 1 year ago • 4 comments

mhd-fettah avatar Mar 23 '23 18:03 mhd-fettah

thanks, i will make a test later

josStorer avatar Mar 24 '23 02:03 josStorer

no thank youuuu bro for accepting contributions from us . I will keep helping there is many stuff in my mind to improve the plugin since I use it every day now .

mhd-fettah avatar Mar 24 '23 07:03 mhd-fettah

I have conducted some tests and noticed that numerous styles and functions appear to be compromised. It might be beneficial for you to perform your own build tests to ensure everything is working as intended.

There are subtle differences between the icons in @primer/octicons-react and react-bootstrap-icons. In my opinion, the download button in @primer/octicons-react seems to be more visually appealing.

Please note that the icons in @primer/octicons-react do not support onClick. As a result, I utilized a span to achieve the desired functionality. It is essential not to remove this part, or the functionality will be lost.

The gpt-util-icon style is solely for changing the cursor to indicate that the element is clickable. This style is being used in many places, and adding extra styles to it has inadvertently caused disruptions in the appearance of other buttons.

josStorer avatar Mar 24 '23 08:03 josStorer

I will try to build and check whats broken and fix it . we need to standardize and improve the UI & UX so the plugin can bypass others in term of approval for final user .

mhd-fettah avatar Mar 28 '23 21:03 mhd-fettah