nextcloud / dashboard

ARCHIVED, new Dashboard is in the server
https://nextcloud.com/dashboard
GNU Affero General Public License v3.0
58 stars 29 forks source link

update for nextcloud v14 and v15 #46

Closed tuxedo-rb closed 5 years ago

tuxedo-rb commented 5 years ago

this pull request fixes the layout for nextcloud v14 (which refers to open issue #45), removes the unused db-table 'dashboard_files', adds theming app color for the section bottom borders, if theming app is installed (which refers to open issue #19) and some minor fixes.

ArtificialOwl commented 5 years ago

I appreciate your time spent on this 'almost-deprecated' app. I will have a look to this as soon as possible

tuxedo-rb commented 5 years ago

No hurry.

Hmm, it seems Nextcloud v14 messed up more than only the layout. The tinymce editor won't open to create/edit announcements. So the announcement section is totally unusable. And if i click on a event in fullcalendar, no overlay opens (well, minor issue). I'm not a javascript expert, so i don't know, how to fix this.

Does Nextcloud14 handle third-party javascript in a different way now?

tuxedo-rb commented 5 years ago

Ok, i've found the problem. The main content overlaid the ui-elements. I increased the css property 'z-index' for ui-elements.

ArtificialOwl commented 5 years ago

Do you see any issue with merging your work on stable14 and release it as a 14-only version ?

tuxedo-rb commented 5 years ago

That's ok for me.

tuxedo-rb commented 5 years ago

My latest commit fixes also issue #18. I replaced the sql function curdate() with a php generated date.

tuxedo-rb commented 5 years ago

I close this merge request, due to the age and changes on a publicly unused stale branch.