Collabora Online Weekly Meeting #159
Date: Mar 7th 2024
All minutes published on: Topics tagged meeting-minutes
Please update your section
Please update your section or create a new one if needed.
- Testing
- Calc Testing: Nextcloud pass: coolmeeting
Attendees (write your name):
- Andras, Méven, Ezinne, Darshan, Hub, Caolán, Pedro, Mike D, Anna, Szymon, Skyler, Meven, Bayram, Timur, Attila, Cosmin, Gulsah, Cor, Lars Hoeger, Gökay, Lily, Vivek, Michael, Nicolas
Making the call shorter
- Please update your section relating in the document before we read through it.
- Then only go over anything we need to share / get input on
Release Schedule
- CODE Beta 24.04 some delays here
- Working on it
- Mobile releases
- Not well-planned
Nicolas
- Nothing to share
Cosmin
- COOL Days
- Talk’s duration?
- Probably best to stick with 5 (default) to 10 min talk.
- COOL Days: Strive to have many short cool talks
- Template for slides is coming …
- browser: add last modification and document status at the bottom by gmasei11 · Pull Request #4531 · CollaboraOnline/online · GitHub
- would love to have that polished & merged …
- that is running live (Cosmin)
Lars Hoeger
- Univention chap, heavily involved in openDesk for the last year.
- Giant document profile is fun:
- especially a long document – each time a page is added,
we get this invalidation – where everything is discarded & re-fetched. - Casually editing it – multiple pages appear – when they shouldn’t which is odd
- Tables - split, red-lining – is growing when it shouldn’t be growing
- Some sort of layout bug; find & fix that.
- Hope to bisect this page growth on small changes
- AI: find someone in support to help out with the bisection (Lily/Anna)
- especially a long document – each time a page is added,
Antranig Vartanian
- Representing FreeBSD community
- Trying to pick WRT FreeBSD port
- Wants to contribute any FreeBSD work
- Andras Timar: Another contributor was working on that → Gleb P. was the community maintainer
- So we ended up disabling the Cirrus CI (FreeBSD) because the service was broken
- Antranig: it seems the image you were using was removed from Cirrus community
Julius
- Open in new tab by default (?)
- No news on internal discussions yet
- Pedro: It was agreed to have it as a setting
- No news on internal discussions yet
- WASM
- For now only in development branch of richdocuments, targetting Nextcloud 29
- Discussion with Pedro around open locally
- General problem that we have no mechanism to “inform” Collabora about lock/permissions changes from the Nextcloud server
- Notify Collabora around changed document metadata · Issue #3421 · nextcloud/richdocuments · GitHub
- Ongoing discussion, probably continued next week with Michael
- Very large file access? (Michael)
- WOPI end-point not supporting range requests (Julius)
- is done for the webdav endpoints.
- WOPI end-point not supporting range requests (Julius)
- Issue affecting Android users now fixed and released in the latest RC: Releases · nextcloud/android · GitHub
- Collabora not loading existing documents in app v3.27.0 · Issue #12338 · nextcloud/android · GitHub and the fix was merged last month: Fix collabora files not loading by JonasMayerDev · Pull Request #12349 · nextcloud/android · GitHub
- Released this week → awaiting for user feedback
- @Nnamani Ezinne Martina was testing: Ezinne: I tested the 3.28.0 version which was released on 13th Feb. Opening documents works using COOL 23.05.9.1
- Reset access token: #3462 : fix: Handle Reset_Access_Token to generate a new token and reload by juliushaertl · Pull Request #3462 · nextcloud/richdocuments · GitHub
- New version of Richdocuments will be released
very soon→ released 3 days ago: chore(release): Bump version to 8.3.2 by juliushaertl · Pull Request #3507 · nextcloud/richdocuments · GitHub chore(release): Bump version to 8.2.5 by juliushaertl · Pull Request #3508 · nextcloud/richdocuments · GitHub
Online activity
- Bugs while in the text document
- Well, I was testing the comment feature, at some point I couldn’t add the comment, then I scrolled down and saw lots of white tiles, the file became very slow for me, people’s mouses behaved strange. I have record (Anna )
- Help > About “Copy” button not working (on Share)
- Chrome 122.0.6261.94 + 23.05.9.2snapshot (git hash: 9831402)
- View > Sidebar + Style dropdown
- Every time you open it, the font for “Clear formatting” + “Shapes” changes.
- Layout > Columns menu seems buggy.
- Upping columns from 1->4, lowering 4->1 leaves numbers in the boxes.
- View > Dark Mode vertical scrollbar, can’t tell where you are in document.
- Looks to be “same color” (light-gray on light-gray).
- Attila
- Working on: PPTX display placeholder in slideshow
159258 – A picture placeholder with a background, imported from PPTX, is not shown when presenting
Found a solution and made a PR:
https://gerrit.libreoffice.org/c/core/+/164365
but still working on review - Started working on a slide transition performance issue:
153162 – 8s delay before being able to transition to slide from empty slide (- preparing animated GIF?)
GIF loading (creating bitmaps) seems to be slow.
- Working on: PPTX display placeholder in slideshow
- PRs without reviewer 3 → now 5
- Collabora Online · GitHub
- PRs for 23.05 should have 2 reviewers
- Old PRs
- Darshan: Refactoring: updateNotebookbar accessibilityDefinations by Darshan-upadhyay1110 · Pull Request #7245 · CollaboraOnline/online · GitHub
- Bayram: use getentropy() for BSD systems by bayramcicek · Pull Request #6211 · CollaboraOnline/online · GitHub
- A.I. (Pedro) to check old open PRs that have reviewer
- Szymon
- Calc view jumping:
- CI passed, merged
- Response for PostMessage Action_Export:
- so integrator will know if it was successful or failed
- Allow external trusted source based on alias groups even if port is defined in coolwsd.xml
- added SDK troubleshoot info
- fixed k8s SDK example yaml so it will work after copy paste (indentation)
- More TypeScript in master:
- removing w2ui, use our menus
- Calc view jumping:
- Darshan
- Move freeze options to view tab and add freeze-panes options to column/row header context menu #8451 => Merged
- Close all dialogs on document idle 8418 => Merged
- added cypress test to check if cell is editable or not after dialog close 0ce96e115ef62f3ec499f463517a0b9801d86e35
- The iOS test fails but seems to be unrelated: Close all dialogs on document idle by Darshan-upadhyay1110 · Pull Request #8418 · CollaboraOnline/online · GitHub
- Andras: I have found the problem. In one of the machines we have one missing dependency and then it fails.
- All recent PR in my section which are merged in master are also being backported to 23.05
- WIP: Various issues with paragraph formatting options in mobile view
- Vivek
- Work in progress - Scroll bar animation on mouse hover event (pause)
- Make chart properties dialogs async: Part 2 => Remaining Few. => Merged. https://gerrit.libreoffice.org/c/core/+/164194
- Conditional formatting: It should override other formatting => WIP
- AI (Pedro) Chase conditional format icon related bug (zooming )
- Michael / Caolan – Performance
- Background saving for 24.04
- thread count can be got down to one thread no problem
- several re-factors merged already
- can verify we are one-thread, and opportunistically fork
- fork works – save works in background; but … having two Kit processes drive the same sockets is … exciting =)
- working on closing those in the save process and having a child websocket to control the save process.
- Allow calc to have inline spelling on in one view and off in another
- https://gerrit.libreoffice.org/c/core/+/164106 (merged 24.04)
- calc comment perf improvement with thanks to Kohei Yoshida
- https://gerrit.libreoffice.org/c/core/+/164021 (merged 24.04)
- tdf#160056 working on some calc threading performance improvements
- scope of MutexGuard can be reduced https://gerrit.libreoffice.org/c/core/+/164189
- avoid SvNumberFormatter lock in formula-group-threading https://gerrit.libreoffice.org/c/core/+/164227
- comphelper::rng takes a mutex for every random number https://gerrit.libreoffice.org/c/core/+/164382
- Background saving for 24.04
- Testing results:
- Dark mode now working nicely.
- Found some good problems from calc:
- https://github.com/CollaboraOnline/online/isues/6893 – getTextSelection
- may be a red-herring.
- https://github.com/CollaboraOnline/online/isues/6893 – getTextSelection
- Bayram
- (In Progress): Bug (Enhancement) “Support Power Query in XLSX”
- Bugzilla: 158857 – Keep Power Query when saving XLSX
- Patch (WIP) : https://gerrit.libreoffice.org/c/core/+/164290 (Mentor: Caolán)
- sent 3 patchsets
- Status : Implementing the feature.
- Implement step by step. (for each missing files, references, ids and tags)
- Missing files/folders: xl/connections.xml , xl/queryTables/ *, xl/tables/_rels/ *, (and some references, ids and tags)
- Now, Calc can write missing xml element into [Content_Types].xml file without checking whether xl/connections.xml file exists or not.
- needs more work to export it correctly.
- check if xl/connections.xml file exist.
- next: export xl/queryTables/* references into [Content_Types].xml
- (In Progress): Bug (Enhancement) “Support Power Query in XLSX”
- Skyler
- Done:
- Make JSDialogs fully reachable when wider than the screen by Minion3665 · Pull Request #8470 · CollaboraOnline/online · GitHub
- Fix for small screens with wide JSDialogs
- Also makes JSDialog movement nicer at the edge of the screen
- Make JSDialogs fully reachable when wider than the screen by Minion3665 · Pull Request #8470 · CollaboraOnline/online · GitHub
- Current:
- Pinch-zoom movement issues
- Worked on this for a while, looked at what zoom should be like and what we would need to do that
- Waiting for pointers from Szymon
- Following pill zoom
- Investigating this
- Speculation: Looks like it may be a problem from mixing em and px?
- calc: fix header context menu on touch devices by Minion3665 · Pull Request #8311 · CollaboraOnline/online · GitHub
- Context menu fixes for mobile row/column headers in calc
- I have a solution, but we don’t like it because it introduces hammer.js into a place where it could cause a mess
- Waiting for input from Gokay on how to proceed (possibly without hammer.js?)
- Pinch-zoom movement issues
- Done:
- Pranam
- Done:
- Visual feedback for operations of tracked comments (show/hide changes, indication of delete)
- Current:
- Moving shapes in impress while editing texts inside
- Fixing issues with comments layout in writer
- Done:
- Gökay
- Current:
- The first commits for read only Online features are in.
- Now working on follow up tasks.
- The first commits for read only Online features are in.
- Current:
- Hubert
- Working on a Drupal integration
- Gulsah
- Impress bullet list: We need to prevent to add empty lines. Instead close the numbering
- It should be done it in core. Writer has same behaviour, but doesn’t share the same codebase.
- I’m progressing, about to finish.
- Impress bullet list: We need to prevent to add empty lines. Instead close the numbering
- Andras
- No update
- Pedro
- Annotation/comment fix (odd width)
- Fix sheet tab misalignment
- Fix tooltips order in the stack
- Neil
- Looking at tinderbox failures
- Fixed a horrible unit-test problem with racing between
kit_spare startup and tests - hopefully a long-term stability win for unit-test parallelisation
- Fixed a horrible unit-test problem with racing between
- Looking at tinderbox failures
- Timur
- Interested in knowing the status of Open with local editor fix from richdocuments. What happens when automatic backport fails
- A.I. (Pedro) to ping Julius
- Méven
- MR for impress autoplay ppsx/pps support in review
- https://gerrit.libreoffice.org/c/core/+/163652 adding unit-test
- Impress: Allow to autostart in presentation mode by meven · Pull Request #8321 · CollaboraOnline/online · GitHub ready
- will add documentation for impress &startPresentation
- documentation for url query parameters url merged, Advanced integration — SDK https://sdk.collaboraonline.com/ documentation
- working on zimbra integration
- MR for impress autoplay ppsx/pps support in review
Forum (Mike D)
9 posts done this week.
3 Key topic:
- Nextcloud connects fine to CODE docker container but can't open any documents - #6 by Tex
- User’s specific Nextcloud + Docker + Apache broken.
- nginx worked, apache didn’t.
- (Maybe documentation needs updating? Still don’t know his exact issue though. Lots of version/debug info in topic though.)
- How to insert vector images into collabora - #2 by Tex
- Insert > Image > Insert Image Can’t Choose SVG. (Like you can with JPG, PNG, etc.)
- On Share, I don’t see it. There’s only an “Insert > Image” button. No secondary menu.
- We just get a URL → they should file it with their integration: Nextcloud, ownCloud etc. (Michael)
- Insert > Image > Insert Image Can’t Choose SVG. (Like you can with JPG, PNG, etc.)
- Beginner questions // Where do I start / first steps as a Collabora - Nextcloud user
- User came in with many questions on :
- Templates
- Tables (+ Table Styles?)
- “Simple” Layouts (+ Frame Styles?)
- User came in with many questions on :
and updates on previous topic too:
- Question About Display Options/Settings On Android Device - #6 by Tex
- Android + Impress: “Fullscreen Presentation” seems to be forced landscape.
- Doesn’t follow Android’s Auto-Rotate / Rotation settings.
9 → 9 - zero replies in user support:
→ User Support - Collabora Online
37 → 37 - zero replies in installation and config:
→ Installation & Configuration - Collabora Online
Next events
- COOL Day 2024 – April 8 to 10, 2024, Cambridge, UK – COOL Days 2024 - 8-10 April - Cambridge, UK - Collabora Office and Collabora Online
- We are thrilled to announce COOL Days 2024, another exciting and enriching conference and team building event in the picturesque city of Cambridge!
- What to do next
- Passport & VISA
- Make sure you have a valid passport for travel (until at least 11th October 2024 to avoid any issues).
- Apply for a VISA if needed.
- Register for the event: https://eventyay.com/e/0512424d
- Submit a talk:
- What to do next
Next meeting
- 14 /03/2024