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

feat: unify header #2808

Closed
wants to merge 2 commits into from
Closed

feat: unify header #2808

wants to merge 2 commits into from

Conversation

luka-nextcloud
Copy link
Contributor

@luka-nextcloud luka-nextcloud commented Aug 18, 2022

Signed-off-by: Luka Trovic [email protected]

Summary

unified-header-update.mov

@vinicius73 vinicius73 added enhancement New feature or request design Experience, interaction, interface, … 3. to review labels Aug 18, 2022
Copy link
Member

@vinicius73 vinicius73 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've sent some comments.

Please, check the ESLint warnings

src/components/Menu/MenuBar.vue Outdated Show resolved Hide resolved
src/components/Menu/MenuBar.vue Show resolved Hide resolved
src/components/Menu/MenuBar.vue Outdated Show resolved Hide resolved
src/helpers/files.js Outdated Show resolved Hide resolved
src/components/Menu/ActionEntry.scss Outdated Show resolved Hide resolved
@juliusknorr juliusknorr added this to the Nextcloud 26 milestone Oct 11, 2022
@luka-nextcloud luka-nextcloud self-assigned this Jan 10, 2023
@cypress
Copy link

cypress bot commented Jan 11, 2023



Test summary

86 30 0 0Flakiness 1


Run details

Project Text
Status Failed
Commit a81c8c3
Started Jan 11, 2023 3:06 PM
Ended Jan 11, 2023 3:17 PM
Duration 11:26 💡
OS Linux Ubuntu -
Browser Electron 106

View run in Cypress Dashboard ➡️


Failures

workspace.spec.js Failed
1 Workspace > formats text
2 Workspace > creates headings via submenu
3 Workspace > creates lists
4 Workspace > emoji picker
5 Workspace > relative folder links
6 Workspace > callouts > create callout
7 Workspace > callouts > toggle callouts
attachments.spec.js Failed
1 Test all attachment insertion methods > [share] check everything behaves correctly on the share target user side
shortcuts.spec.js Failed
1 keyboard shortcuts > bold
2 keyboard shortcuts > italic
This comment includes only the first 10 test failures. See all 30 failures in the Cypress Dashboard.

Flakiness

cypress/e2e/conflict.spec.js Flakiness
1 Open test.md in viewer > displays conflicts

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@luka-nextcloud
Copy link
Contributor Author

@juliushaertl I've uploaded a video to show the status.

@juliusknorr
Copy link
Member

@luka-nextcloud I think we agreed on trying to use portal-vue as a way to not re-implement the header bar, but instead try to only inject the menubar parts into the existing header.

#82 (comment)

If there have been any obstacles, maybe you can provide a summary of what you tried and why this didn't work.

This would be a flexible way to also take the mockups into account which are different for the logged in users where the viewer is used with the white header bar and the public share links that use the blue existing bar. We should try to avoid reimplementing the parts that are already in there by default like the close button/actions/download button.

@juliusknorr
Copy link
Member

@luka-nextcloud Let me close this for now as it is on hold and needs a different approach anyways as we talked about the possible usage of vue-portal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review design Experience, interaction, interface, … enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unified header
4 participants