Skip to content

Commit

Permalink
Merge pull request #850 from FlowFuse/debug-order
Browse files Browse the repository at this point in the history
Remove page order injected for debugging
  • Loading branch information
joepavitt authored May 14, 2024
2 parents 2149759 + d2a2473 commit 83c828e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions cypress/tests/config-nodes/pages.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@ describe('Node-RED Dashboard 2.0', () => {

// Check we have the pages listed
cy.get('[data-nav="dashboard-ui-page-1"]').should('be.visible')
cy.get('[data-nav="dashboard-ui-page-1"]').should('have.text', 'Page 1')
cy.get('[data-nav="dashboard-ui-page-2"]').should('be.visible')
cy.get('[data-nav="dashboard-ui-page-2"]').should('have.text', 'Page 2')

// Click page 2
cy.get('[data-nav="dashboard-ui-page-2"]').click({ force: true }) // for some reason Cypress thinks this element is off the screen - it isn't
Expand Down
2 changes: 1 addition & 1 deletion ui/src/layouts/Baseline.vue
Original file line number Diff line number Diff line change
Expand Up @@ -224,7 +224,7 @@ export default {
}
},
getPageLabel (page) {
return page.order + ' ' + page.name + (this.dashboard.showPathInSidebar ? ` (${page.path})` : '')
return page.name + (this.dashboard.showPathInSidebar ? ` (${page.path})` : '')
},
handleNavigationClick () {
if (this.navigationStyle === 'fixed') {
Expand Down

0 comments on commit 83c828e

Please sign in to comment.