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

Widget fixes #3856

Merged
merged 4 commits into from
Mar 2, 2023
Merged

Widget fixes #3856

merged 4 commits into from
Mar 2, 2023

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Mar 1, 2023

  • fix: Limit paragraph spacing to the content
  • fix: Only attempt to show reference widgets for logged in users

📝 Summary

Avoid rendering issues with reference previews on paragraphs and public share links.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@cypress
Copy link

cypress bot commented Mar 1, 2023

Passing run #8809 ↗︎

0 138 0 0 Flakiness 0

Details:

Widget fixes
Project: Text Commit: af1a55c607
Status: Passed Duration: 03:15 💡
Started: Mar 2, 2023 11:28 AM Ended: Mar 2, 2023 11:32 AM

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

@juliusknorr juliusknorr added bug Something isn't working 3. to review labels Mar 1, 2023
Copy link
Member

@mejo- mejo- left a comment

Choose a reason for hiding this comment

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

Code looks good to me.

@mejo-
Copy link
Member

mejo- commented Mar 1, 2023

I wonder though why the Node CI job didn't fail - this PR contains changes to the JS sources that should require a compile, no? 🤔

@juliusknorr
Copy link
Member Author

Indeed, I first had suspected nextcloud/.github#124 but we only have js/scss files here.

@juliusknorr
Copy link
Member Author

Ah I think it is because css/* changes are also not part of the path checks for node. :/

@juliusknorr
Copy link
Member Author

/compile

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr juliusknorr force-pushed the bugfix/noid/widget-paragraph branch from c8d4b68 to 734e144 Compare March 2, 2023 11:07
@juliusknorr
Copy link
Member Author

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@juliusknorr juliusknorr merged commit 7c3a6ee into main Mar 2, 2023
@delete-merged-branch delete-merged-branch bot deleted the bugfix/noid/widget-paragraph branch March 2, 2023 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants