Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(settings): overhaul settings layout #1412

Merged
merged 9 commits into from
Apr 10, 2021
Merged

fix(settings): overhaul settings layout #1412

merged 9 commits into from
Apr 10, 2021

Conversation

JPSchellenberg
Copy link
Member

Hey,

before I start to integrate the library-management I have to overhaul the settings-section to make it more structured as we introduce more and more settings.

Let me know what you think.. I will add the translations (and the implementation of the account- & library-settings) later.

@JPSchellenberg JPSchellenberg marked this pull request as ready for review April 5, 2021 21:36
@sr258
Copy link
Member

sr258 commented Apr 6, 2021

I like the way this is going! A few remarks:

  • Users should not be able to withdraw consent to necessary data processing, as it makes no sense. The app is unusable then, so this setting should not exist.
  • the view on the right-hand side (details-view / main area with all the settings) has too large horizontally: I can scroll it.
  • the title of the "General" tab should also be "General" in the title of the actual settings. Currently it is "settings".
  • It would be good if the currently selected tab was highlighted on the left.

image

@JPSchellenberg
Copy link
Member Author

I fixed some issues you mentioned and reworked the general section and divided it up to a general and updates section as I plan to improve the update process, which will require a separate updates section.

@JPSchellenberg JPSchellenberg merged commit cb513cd into master Apr 10, 2021
github-actions bot pushed a commit that referenced this pull request Apr 17, 2021
# [0.7.0](v0.6.1...v0.7.0) (2021-04-17)

### Bug Fixes

* **analytics:** improve error-handling ([#1443](#1443)) ([287c675](287c675))
* **analytis:** increase search bar to fit text ([c146247](c146247))
* **cancelExport:** use cancelExportH5P-action consistently ([d0dc8c7](d0dc8c7))
* **client:** updated Spanish locales ([8e1111a](8e1111a))
* **deps:** update dependency @testing-library/user-event to v13.1.2 ([d7d3561](d7d3561))
* **deps:** update dependency @testing-library/user-event to v13.1.3 ([26520b9](26520b9))
* **deps:** update dependency @types/node to v14.14.39 ([97236f2](97236f2))
* **deps:** update dependency @types/node to v14.14.41 ([cd7d325](cd7d325))
* **deps:** update dependency classnames to v2.3.1 ([d94d87c](d94d87c))
* **deps:** update dependency electron-log to v4.3.3 ([2358356](2358356))
* **deps:** update dependency electron-log to v4.3.4 ([9646ffc](9646ffc))
* **deps:** update dependency i18next to v20.2.0 ([0470714](0470714))
* **deps:** update dependency i18next to v20.2.1 ([e5148a4](e5148a4))
* **deps:** update dependency i18next-http-middleware to v3.1.1 ([a344d1c](a344d1c))
* **deps:** update dependency react-i18next to v11.8.13 ([b6eb1b3](b6eb1b3))
* **deps:** update dependency typescript to v4.2.4 ([80077f8](80077f8))
* **devmode:** empty --libs switch ignored ([9fa6e3b](9fa6e3b))
* **devmode:** pass undefined if libs is not set ([b5e91a7](b5e91a7))
* **general:** added missing default Mac menu ([d3388e2](d3388e2))
* **i18n:** Translated using Weblate (Spanish (Mexico) (es_MX)) ([#1447](#1447)) ([653410e](653410e))
* **locales:** auto-translated new strings ([02552ce](02552ce))
* **locales:** corrected auto-translated German locale ([8b3bbc7](8b3bbc7))
* **locales:** run autotranslation ([5dde3e1](5dde3e1))
* **locales:** Translations update from Weblate ([88456da](88456da))
* **mac-menu:** missing default mac-menu ([961abbf](961abbf))
* **merge:** fixed incorrect merge ([5739f6d](5739f6d))
* **sentry:** add new DSN ([#1438](#1438)) ([0228646](0228646))
* **server:** improved file dialogs ([5b13df9](5b13df9))
* **settings:** overhaul settings layout ([#1412](#1412)) ([cb513cd](cb513cd))

### Features

* **auth:** add auth via passwordless login & register ([#1410](#1410)) ([d5e31af](d5e31af))
* **client:** added cancel to export dialog ([843e3cd](843e3cd))
* **client:** added export options ([2c7379b](2c7379b))
* **client+server:** added SCORM mastery score ([6952d5b](6952d5b))
* **editor:** content type names in tab headers now localized ([#1419](#1419)) ([eed23e8](eed23e8))
* **editor:** dev mode ([#1432](#1432)) ([d6035e5](d6035e5))
* **exports:** added more export options ([1d9c95b](1d9c95b))
* **exports:** updated locales ([f2b6bf7](f2b6bf7))
* **library-management:** add h5p-library-management to settings ([#1416](#1416)) ([1e83648](1e83648))
* **mac menu:** added localization ([9d2917a](9d2917a))
* **server:** changed export ([0311a34](0311a34))
* **server:** external export works ([b892d1f](b892d1f))
* **server:** first functional SCORM exporter ([2ffd2f5](2ffd2f5))
* **server+client:** localized export ([3e65a45](3e65a45))
@github-actions
Copy link

🎉 This PR is included in version 0.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@JPSchellenberg JPSchellenberg deleted the fix/settings branch May 14, 2021 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants