UI Bugs + Suggestion - Githubissues

dbeck121 / CPI-Helper-Chrome-Extension

53 stars 18 forks source link

UI Bugs + Suggestion #107

Closed incpi closed 9 months ago

incpi commented 10 months ago

Hello 3.8.1 has few Minor UI issues,

Color of sidebar looks awful under certain scenarios.
Popup Text is too large then expected on small Screens.( Zoom levels might need or something similar. )

Plugin UI (Original Message) I would appreciate it if you could share your feedback on this or any changes that you think are necessary.

fippu82 commented 10 months ago

Looks good to me. What I probably would change is the two labels for version and id and put the version maybe to the very right of the blue title bar and the ID small to the bottom right (it's unimportant for the user) and no gray background for both.

dbeck121 commented 10 months ago

looks good! Big improvements to the old UI.... Do you use the fomantic ui css that is already included in CPI Helper?

incpi commented 10 months ago

Hello dbeck121, I do agree with fippu82. Can't we ditch the Version ID & plugin ID because every plugin will be added to git then it moves to store? So, I don't think any point of adding it for normal user (to move elsewhere, I will check and let see how it goes)?. Yes, I built on top of fomantic UI with additional custom CSS.

dbeck121 commented 10 months ago

Fine with me

incpi commented 9 months ago

Hello @dbeck121 ,

Small Suggestion because when i started this i face this issue which file do what where my code goes. So i think this will be help full to maintain and find thing easily.

File structure diagram

Old Layout (left) current. New Layout (right)

dbeck121 commented 9 months ago

Would be fine with me. It gives a way better structure.

fippu82 commented 9 months ago

Hi both, just one remark: imo the file logs.js isn't "common" as it is not actually related to the CPI Helper logging but to the logs view for CPI messages, if I'm not mistaking. So it should rather be put into "scripts".