BetterChatGPT icon indicating copy to clipboard operation
BetterChatGPT copied to clipboard

Move low-frequency options to settings menu

Open solstice23 opened this issue 3 years ago • 5 comments

  • Move low-frequency options to settings menu
  • ~~Add lockfiles to .gitignore~~ image image

solstice23 avatar Mar 14 '23 15:03 solstice23

Thank you for the reminder. I have reverted the .gitignore change and moved the theme switcher to the settings menu. However, there was a new package-lock.json generated because I used npm. Should I delete this file?

solstice23 avatar Mar 14 '23 16:03 solstice23

You should not include package-lock.json in the commit.

ztjhz avatar Mar 15 '23 08:03 ztjhz

The cursor for the import/export and clear conversation buttons is not pointer. Add cursor-pointer to their className.

image

ztjhz avatar Mar 15 '23 13:03 ztjhz

As "Clear Conversation" and "Import/Export" both pertain to managing the chat history, whether by clearing, exporting, or importing, they should, in my opinion, be kept (grouped) together, not separated, whether they are placed in the sidebar menu or in the settings submenu.

SamuelMiller avatar Mar 15 '23 15:03 SamuelMiller

As "Clear Conversation" and "Import/Export" both pertain to managing the chat history, whether by clearing, exporting, or importing, they should, in my opinion, be kept (grouped) together, not separated, whether they are placed in the sidebar menu or in the settings submenu.

They have already moved to the settings menu.

solstice23 avatar Mar 15 '23 23:03 solstice23

With the new foldable menu feature for the settings in the latest BetterChatGPT, I don't think we need to move the low-frequency options as the user can simply just collapse the menu to make more space for the chat history. My original request offered two suggested solutions to make more room for the settings. Now with the collapsable menu feature my request has been fulfilled.

SamuelMiller avatar Mar 26 '23 15:03 SamuelMiller