Repository navigation
feat(applications): overview page to manage pages and menu - #53
Open
rubenvdlinde wants to merge 1 commit into
Open
rubenvdlinde wants to merge 1 commit into
rubenvdlinde wants to merge 1 commit into
Conversation
Inside Tables an application gets an overview at /application/{id}/overview,
linked from its title row: its address, settings, pages and menu. The pages
come from the application (start page first), with their widgets and, on the
start page, its tables and views. "Add page" uses the page form, adds the page
to the menu and opens it in edit mode. The menu has a numbered preview and is
edited in place with the editor of the create dialog, which now offers the
pages of the application.
Assisted-by: ClaudeCode:claude-opus-5-5
This was referenced Oct 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rebuild of #40 (originally #26) on the page model of nextcloud#3100 / nextcloud#3099, see #13. Stacked on #52.
What changes
Inside Tables an application gets an overview to manage it: its name, address, settings, pages and menu.
/application/{id}/overview, opened with "Overview" in the application's title row.?edit=1, newstartInEditModeprop onGridPage), others "Open".Carried over from #40, and what changed because of nextcloud#3100 / nextcloud#3099
toEditableMenuItems,pageTitle,updateContextMenuItems: carried over. In the editor, feat(applications): overview page to manage pages and menu聽#40'sisFixedItembecame m4'sopensNewStartPage./application/{id}as it is, with the page bar under the description. feat(applications): overview page to manage pages and menu聽#40 replaced that route with the overview; here the overview has its own route and the start page stays. For the same reason the Tables navigation still collapses when an application opens (feat(applications): overview page to manage pages and menu聽#40 had changed that), and the existingcontext*.spec.tsneed no change.pages: title, emoji, page type, technical name); the overview loads each page from the page API for its widget count. feat(applications): overview page to manage pages and menu聽#40 countedpage.contentrows as widgets.POST .../pageswith title, emoji, description and technical name.Tests
Nextcloud 34 container (PostgreSQL 16), Playwright at 1600 x 1000,
--timeout=120000:application.spec.ts: the 4 tests of feat(applications): standalone application shell with its own menu聽#52 plus: the overview lists the start page ("Dashboard", "Start page", "0 widgets") and a page added there gets a menu entry (intake_home) and opens in edit mode; menu entries reordered and nested with buttons keep their place after a reload; a menu entry pointed at a page of the application opens it. Withcontext-menu.spec.ts:9 passed (1.8m).context-grid-pages.spec.ts,context.spec.ts,context-navigation.spec.ts:15 passed (7.6m).env -u ELECTRON_RUN_AS_NODE npx cypress run --component --spec cypress/component/menuItems.cy.js,cypress/component/gridLayout.cy.js,cypress/component/GridWidget.cy.js:All specs passed! 31 31(addstoEditableMenuItemsandpageTitlefor context and page API pages).npm run lint: 0 errors, 1 inherited warning (src/store/store.js:176);npm run stylelint: clean;npm run build: OK.Prepared in the Conduction fork for porting upstream by @remko48. Prepared with Claude Code.