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.

Sign up for GitHub

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

Jump to bottom

IBX-662: Uncaught TypeError: Cannot read property 'getBoundingClientRect' of null at setContainerHeight #1786

Merged
merged 1 commit into from
Jun 28, 2021
Merged

IBX-662: Uncaught TypeError: Cannot read property 'getBoundingClientRect' of null at setContainerHeight #1786

merged 1 commit into from
Jun 28, 2021

Conversation

GrabowskiM
Copy link
Contributor

Question Answer
Tickets https://issues.ibexa.co/browse/IBX-662
Bug fix? yes
New feature? no
BC breaks? no
Tests pass? yes
Doc needed? no
License GPL-2.0

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@bogusez
Copy link
Contributor

bogusez commented Jun 28, 2021

@GrabowskiM cannot publish later content "Landing page"

Uncaught TypeError: Cannot read property 'removeAttribute' of null at focusOnFirstError

Screenshot 2021-06-28 at 14 47 54

@GrabowskiM
Copy link
Contributor Author

That's a different one bug, not connected with the one this PR fixes. Could you create new jira issue for this one? I'll try to look at it.

@bogusez
Copy link
Contributor

bogusez commented Jun 28, 2021 •
edited
Loading

That's a different one bug, not connected with the one this PR fixes. Could you create new jira issue for this one? I'll try to look at it.

Yes, sure. I will create a new task.

@dew326
Copy link
Member

dew326 commented Jun 28, 2021

Also, this console error doesn't prevent from publishing content. You have not filled fields "title" and "description".

@lserwatka lserwatka merged commit 61e79b9 into 2.3 Jun 28, 2021
@lserwatka lserwatka deleted the IBX-662-footer-fix branch June 28, 2021 13:51
@lserwatka
Copy link
Member

You can merge it up now.

@GrabowskiM
Copy link
Contributor Author

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tischsoic tischsoic tischsoic approved these changes

@dew326 dew326 dew326 approved these changes

@lucasOsti lucasOsti lucasOsti approved these changes

@bogusez bogusez bogusez approved these changes

Assignees
No one assigned
Labels
QA approved
Milestone
No milestone
6 participants
@GrabowskiM @bogusez @dew326 @lserwatka @tischsoic @lucasOsti