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

Adjust smart picker menu action #4063

Merged
merged 3 commits into from
Apr 17, 2023

Conversation

julien-nc
Copy link
Member

@julien-nc julien-nc commented Apr 17, 2023

  • Adjust the menu item label
  • Adjust the icon to use the same as in Mail

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc added design Experience, interaction, interface, … 3. to review integration labels Apr 17, 2023
@julien-nc julien-nc added this to the Nextcloud 27 milestone Apr 17, 2023
@julien-nc
Copy link
Member Author

IMO this can be backported to stable26 but this is not a bug fix...

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>
@julien-nc julien-nc force-pushed the enh/noid/smart-picker-menu-action-label branch from 5750b9d to ded5e02 Compare April 17, 2023 10:49
@julien-nc
Copy link
Member Author

/compile

@julien-nc julien-nc changed the title Adjust smart picker menu action label Adjust smart picker menu action Apr 17, 2023
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@cypress
Copy link

cypress bot commented Apr 17, 2023

3 flaky tests on run #9429 ↗︎

0 142 1 0 Flakiness 3

Details:

Adjust smart picker menu action
Project: Text Commit: 53e89eb07b
Status: Passed Duration: 04:33 💡
Started: Apr 17, 2023 10:59 AM Ended: Apr 17, 2023 11:04 AM
Flakiness  sync.spec.js • 1 flaky test

View Output Video

Test Artifacts
Sync > recovers from a lost connection Output Screenshots
Flakiness  conflict.spec.js • 2 flaky tests

View Output Video

Test Artifacts
Open test.md in viewer > resolves conflict using current editing session Output Screenshots
Open test.md in viewer > resolves conflict using server version Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@max-nextcloud
Copy link
Collaborator

/backport to stable26

@max-nextcloud
Copy link
Collaborator

IMO this can be backported to stable26 but this is not a bug fix...

I'll create a backport request. Stable26 is blocked for most of the week anyway and we can discuss if we want to backport in the backport PR.

@max-nextcloud max-nextcloud merged commit bb16978 into main Apr 17, 2023
@delete-merged-branch delete-merged-branch bot deleted the enh/noid/smart-picker-menu-action-label branch April 17, 2023 18:08
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, … integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants