Skip to content

Conversation

@PVince81
Copy link
Member

In some situations the element is still affecting the layout, especially
when a sticky is nearby.

This fix moves it properly off-screen.

There is a risk that this might break existing layouts that were relying on this, if any.

In some situations the element is still affecting the layout, especially
when a sticky is nearby.

This fix moves it properly off-screen.

Signed-off-by: Vincent Petry <[email protected]>
@PVince81 PVince81 added bug design Design, UI, UX, etc. 3. to review Waiting for reviews labels Oct 29, 2020
@PVince81 PVince81 added this to the Nextcloud 21 milestone Oct 29, 2020
@PVince81 PVince81 requested a review from skjnldsv October 29, 2020 20:38
@PVince81 PVince81 self-assigned this Oct 29, 2020
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Looks good, make sense :)

@PVince81
Copy link
Member Author

Ref for the place it was discovered in the Talk app: nextcloud/spreed#4493 (comment)

@faily-bot
Copy link

faily-bot bot commented Oct 30, 2020

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 34787: failure

mariadb10.4-php7.4

Show full log
There was 1 error:

1) OCA\DAV\Tests\unit\CalDAV\CalDavBackendTest::testMultiCalendarObjects
TypeError: Argument 2 passed to OCA\DAV\Events\CalendarCreatedEvent::__construct() must be of the type array, null given, called in /drone/src/apps/dav/lib/CalDAV/CalDavBackend.php on line 790

/drone/src/apps/dav/lib/Events/CalendarCreatedEvent.php:51
/drone/src/apps/dav/lib/CalDAV/CalDavBackend.php:790
/drone/src/apps/dav/tests/unit/CalDAV/AbstractCalDavBackend.php:153
/drone/src/apps/dav/tests/unit/CalDAV/CalDavBackendTest.php:314

--

There were 2 warnings:

1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

mysql5.6-php7.3

Show full log
There was 1 error:

1) OCA\DAV\Tests\unit\CalDAV\CalDavBackendTest::testPurgeAllCachedEventsForSubscription
TypeError: Argument 2 passed to OCA\DAV\Events\SubscriptionCreatedEvent::__construct() must be of the type array, null given, called in /drone/src/apps/dav/lib/CalDAV/CalDavBackend.php on line 2117

/drone/src/apps/dav/lib/Events/SubscriptionCreatedEvent.php:51
/drone/src/apps/dav/lib/CalDAV/CalDavBackend.php:2117
/drone/src/apps/dav/tests/unit/CalDAV/AbstractCalDavBackend.php:173
/drone/src/apps/dav/tests/unit/CalDAV/CalDavBackendTest.php:963

--

There were 2 warnings:

1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

sure

@rullzer rullzer merged commit 3b97819 into master Oct 31, 2020
@rullzer rullzer deleted the bugfix/noid/fix-hidden-visually branch October 31, 2020 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug design Design, UI, UX, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants