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

CRISTAL-219: UI for creating pages #348

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

CRISTAL-219: UI for creating pages #348

wants to merge 2 commits into from

Conversation

pjeanjean
Copy link
Contributor

Jira URL

https://jira.xwiki.org/browse/CRISTAL-219

Changes

Description

  • Add New Page dialog to the sidebar: CPageCreationMenu
  • Make navigation trees react to current page changes
  • Add prop to XNavigationTree to override default click action
  • Introduce NavigationTreeSource.getParentNodesId()
  • Introduce WikiConfig.getPageResourceSeparator()
  • Introduce WikiConfig.getNewPageDefaultName()
  • Introduce property NavigationTreeNode.location
  • Implement missing model on Shoelace's XTextField
  • Implement missing model on Shoelace's XDialog
  • Slightly unify CSS between Shoelace and Vuetify components
  • Update vuetify to version 3.6.15

Clarifications

The changes to the Navigation Tree will only work for the Nextcloud backend after this PR is merged.

vuetify was updated to the last iteration of 3.6.x in order to fix some bugs with VTreeview. Updating to 3.7.x will need to be postponed until vuetifyjs/vuetify#20460 is resolved.

Screenshots & Video

Shoelace
image

Vuetify
image

Executed Tests

Everything was tested manually and the test suite was run succesfully.

Expected merging strategy

  • Prefers squash: Yes
  • Backport on branches:
    • N/A

* Add New Page dialog to the sidebar: `CPageCreationMenu`
* Make navigation trees react to current page changes
* Add prop to `XNavigationTree` to override default click action
* Introduce `NavigationTreeSource.getParentNodesId()`
* Introduce `WikiConfig.getPageResourceSeparator()`
* Introduce `WikiConfig.getNewPageDefaultName()`
* Introduce property `NavigationTreeNode.location`
* Implement missing model on Shoelace's `XTextField`
* Implement missing model on Shoelace's `XDialog`
* Slightly unify CSS between Shoelace and Vuetify components
* Update `vuetify` to version `3.6.15`
* Fix functional tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant