richdocuments icon indicating copy to clipboard operation
richdocuments copied to clipboard
verified publisher icon nextcloud

feat: introduce pinia for admin settings

Open elzody opened this issue 9 months ago • 4 comments

  • Related to: https://github.com/nextcloud/richdocuments/issues/4351
  • Target version: main

Summary

Introduces Pinia in an effort to organize the admin settings a bit instead of having everything in one massive component.

Screen Recording

Custom Fonts

https://github.com/user-attachments/assets/17145797-2776-4a71-83a1-984516e186fd

TODO

  • [x] Tests

Checklist

  • [x] Code is properly formatted
  • [x] Sign-off message is added to all commits
  • [x] Documentation (manuals or wiki) has been updated or is not required

elzody avatar Mar 24 '25 17:03 elzody

@juliusknorr Do you think we should ignore the remote font config warning for now? Didn't we discuss that, because we can have these configuration files of our own now, we can always ensure there is one?

elzody avatar Apr 01 '25 20:04 elzody

Curently we cannot verify this anyways as it is a setting in coolwsd.xml and we cannot check that.

Mid-term collabora planned to remove this configuration option as far as I understood and pass over the fonts without any configuration required, so I'd say it is fine to ignore that, yes.

juliusknorr avatar Apr 01 '25 20:04 juliusknorr

@nextcloud/designers I have uploaded a screen recording, can I get a design review on this please? :)

elzody avatar Apr 01 '25 23:04 elzody

what would be cool is a nice component for lists so this is consistent across Nextcloud. If we want to get fancy, this could be a grid/tiled view with font and font name below each other, sort of laid out like the tiles we use for the settings of Dashboard.

We already have NcListItem as a consistent component for list items. Probably can do a lot like this. Also this is very specific not sure how often it would be reused, but of course possible to do.

susnux avatar Apr 14 '25 09:04 susnux