-
Notifications
You must be signed in to change notification settings - Fork 159
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
Improve sharing tab style & add icons #6573
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
23c69c2
to
51d6754
Compare
Results for oc10SharingIntUsers3 https://drone.owncloud.com/owncloud/web/23522/31/1 💥 The acceptance tests pipeline failed. The build has been cancelled. |
XPath selector |
Could you include a screenshot in the PR description? |
51d6754
to
3625863
Compare
Done |
3625863
to
5d39455
Compare
Included, good hint, thx! |
@@ -56,17 +61,21 @@ export default { | |||
return routes | |||
}, {}) | |||
const navItems = computed(() => [ | |||
// computed aka not reactive? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dev leftover? computed -> immutable (at least not via setting a value)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Meh, that was my reminder to investigate what later "magically" worked 🤷🏽 removing it
5d39455
to
19acb5e
Compare
c7572da
to
b9f508a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
b9f508a
to
1370c43
Compare
1370c43
to
4587dd0
Compare
Kudos, SonarCloud Quality Gate passed! |
Results for oCISSharingInternal2 https://drone.owncloud.com/owncloud/web/23586/60/1
|
Results for oCISSharingPublic2 https://drone.owncloud.com/owncloud/web/23586/68/1
|
Results for oC10IntegrationApp1 https://drone.owncloud.com/owncloud/web/23586/71/1
|
Related Issue
Screenshots
Desktop/Tablet
Mobile