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

[Nc 14] Sidebar on adding a new event atop of calendar view #876

Closed
blizzz opened this issue Jul 31, 2018 · 1 comment
Closed

[Nc 14] Sidebar on adding a new event atop of calendar view #876

blizzz opened this issue Jul 31, 2018 · 1 comment
Labels
1. to develop Accepted and waiting to be taken care of bug design Related to design, interface, interaction design, UX, etc.

Comments

@blizzz
Copy link
Member

blizzz commented Jul 31, 2018

Steps to reproduce

  1. Have "skip simple event editor" enabled
  2. be on 14 beta
  3. add a new event

Expected behaviour

The event sidebar should appear on the right and decrease the width of the calendar view (as with Nc 13)

Actual behaviour

The event sidebar is put on top, the calendar view is pushed down, below the fold.

Server configuration

Operating system: Ubuntu 16.04

Web server: Apache

Database: MySQL

PHP version: 7.0

Server version: 14.0.0.13

Calendar version: 1.6.1

Updated from an older installed version or fresh install: updated

Signing status (ownCloud/Nextcloud 9.0 and above): 👍

Are you using an external user-backend, if yes which one: LDAP

Client configuration

Browser: FF 61.0.1

Operating system: Antergos

CalDAV-clients: WebUI

Logs

Browser log

Content Security Policy: The page’s settings blocked the loading of a resource at https://cloud.nextcloud.com/apps/files/ (“script-src”).

Deprecation warning: use moment.updateLocale(localeName, config) to change an existing locale. moment.defineLocale(localeName, config) should only be used for creating a new locale See http://momentjs.com/guides/#/warnings/define-locale/ for more info. sentry.js:1:7668

<script> source URI is not allowed in this document: “https://cloud.nextcloud.com/apps/notifications/js/notifications.js?v=59c93372-49”. calendar:44

Attempting to call a FullCalendar method on an element with no calendar.
sentry.js:1:7668
@blizzz blizzz added the bug label Jul 31, 2018
@georgehrke georgehrke added 1. to develop Accepted and waiting to be taken care of design Related to design, interface, interaction design, UX, etc. labels Jul 31, 2018
@georgehrke georgehrke added this to the 1.6.2 milestone Jul 31, 2018
tcitworld added a commit that referenced this issue Aug 6, 2018
Closes #876

I didn't test on NC < 14, but it should be fine since it only puts back
old sidebar behaviour before nextcloud/server#10218

Signed-off-by: Thomas Citharel <[email protected]>
georgehrke pushed a commit that referenced this issue Aug 15, 2018
Closes #876

I didn't test on NC < 14, but it should be fine since it only puts back
old sidebar behaviour before nextcloud/server#10218

Signed-off-by: Thomas Citharel <[email protected]>
@ChristophWurst
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug design Related to design, interface, interaction design, UX, etc.
Projects
None yet
Development

No branches or pull requests

3 participants