Collabora Online Weekly Meeting #179
Date: 25th Jul 2024
All minutes published on: https://forum.collaboraonline.com/tag/meeting-minutes
- Pedro, Bayram, Darshan, Ezinne, Hub, Andras, João, Méven, Anna, Michael, Aron, Caolán, Skyler, Vivek, Gökay, Attila, Szymon, Timur, Gulsah
A.I. Pedro: Remove all the additional paragraph styles for the next meeting
-
Files
-
Features, user flows (Anyone can add here the things they need to test)
-
[Application] [Requester]: what to test
-
[Impress] [Lily]: Basic presentation flows such as creating/removing slides and reordering them
-
[Impress] [Pedro]: Create object shadow and test it in presentation (slide-show) both full screen and in window
-
[Impress] [Pedro]: Resize shape (default line and area color); Resize shape with transparent area and with line set to purple and thickness to 3 pt
-
[Any App] [Darshan]: Dark mode in read-only document. Check the menubuttons works or not. Do they perform operations properly or not ? Read-only shared link
-
Invert background button should be enabled
-
View > Hide menu: should not be there since it doesn’t work (or be disabled)
- Skyler: I suspect this is related to one of my old commits
-
Click in a row/column header right click > non-read-only operations shouldn’t be accessible
-
Help > Server Audit, does nothing and shouldn’t be enabled/visible
-
-
[All apps] [Skyler]: Inverted background in dark mode. Check that it doesn’t share between users …
- [non-calc] [Skyler]: also nice to see if inverted background can save/load properly when you refresh your page. This is known not to work in calc, so don’t bother testing it there…
-
-
CODE (Red) 24.04.6.1 August 1st
-
stuck in release: Aron created the tag but thtaske packages aren’t in the expected place
-
Looking to fix this asap so we can continue he release process
- Mitch is on this.
-
-
Mobile releases 24.04.4 (stable branch) Next week
-
iOS 24.04 Already released
-
Android 24.04 is in test session
- Blocker: android: some parts don’t render at some zoom levels (Rash will look at it) => Blocker is fixed and now being tested by QA team.
-
-
PRs in need for attention (Please add here any PR that needs attention)
-
PRs without reviewer https://github.com/orgs/CollaboraOnline/projects/3/views/2 now 6
-
chore(android): Update Android Gradle Plugin #9546 → added Rashes as reviewer
-
docstate: Modified colorPalettes in docstate to include new color values #9643
- Added Pedro and Gokay as reviewers
-
Added method to disable spell check if document is read-only #9674
- Fixes Spell correction annoyingly turned on for read-only users #7935
-
-
Stalled PRs
- https://github.com/CollaboraOnline/online/pull/9457 Added Szymon as reviewer (as he was already part of the discussion)
-
Nicolas
-
Gabriel
-
Now back
-
We have started to upgrade now to 23
-
Working now on something related to CSV and probably will backport it but I’m waiting for some feedback → https://gerrit.libreoffice.org/c/core/+/165905
-
this was mostly useful for converting CSV → PDF – because separators were not found
-
idea was to detect automatically at conversion.
-
First patch – adds character-set and separator settings
-
can use these when you pass the filter parameters.
-
Instead of separator & character set put ‘detect’
-
-
Would be nice to use it in the import dialog.
-
-
Would love to help out with design for serializing settings (Michael)
-
Julius
-
Getting much more frequent races causing a download of the file … when clicked. (Michael)
-
seems like a richdocuments / Nextcloud race… perhaps more plugins installed …
-
now have draw.io/BPMN/etc. etc. in the list here … probably related to this.
-
Finally have a reproducer on our instance
-
Seems the viewer API has a design flaw in that regard
-
Our files engineering-team is looking into having a work around
-
-
Small bugfix release of richdocuments later (probably today)
-
branding customization fix for logo
-
iframe accessibility
-
Fix issue where files were locked during preview generation with network timeouts which could have lead to failed saves
- There is still a risk, but less likely, currently thinking about better ways to handle this
-
-
Nextcloud 30 upcoming, currently in beta
-
Integration with extract/transform document API
-
Create new documents with form from templates
-
-
Nextcloud Conference is happening Sept 14-15, contributor week afterwards
-
Nightly CODE docker image for easier testing
-
Maybe we should have --enable-experimental in here by default in the online configure line to match what I think we build CODE with by default (Caolán)
-
Thorsten
- [No update]
-
Attila
No update
-
Andras
-
Szymon
-
Following the users
-
Fixed another one with generic approach – we didn’t use correct position for cursor in Writer after enabling new following
-
Calc address bar didn’t jump to the cursor – also fixed
-
-
Reviews
-
Working on Impress and Image insertion from integrator problem
-
Next: Pedro’s report - now unotoolbuttons are being enable/disable but the inner child is not being enabled or disabled: search previous and search next
-
Darshan
-
Fix event binding issue by initializing event handlers once #9660
-
Working on : Fix top document’s margin when rules is on #9613
-
(has local patch) Typescript conversion of Ruler.js. Next will create base class for ruler which will help to remove duplication of code occurring in horizontal and vertical ruler file.
- Maybe we can then hunt why it takes too long to re appear when switching pages
-
Next:
-
More improvements on accessibility stuff
-
add currency drop down in writer app.
-
-
Aron
- nothing to add
-
Caolan performance
-
We now also (attempt to) use linux namespaces if capabilities of false is requested
-
Note: how to attach debugger to a coolforkitns and get working symbols without root
-
https://github.com/CollaboraOnline/online/pull/9493#issuecomment-2237455110
-
(gdb) set sysroot /
-
documented sudo gdb works fine
-
-
Working on extending async DNS
-
Auto committed Watchdog profiles
-
Bayram
-
(In Progress) - Enhancement - tdf#158857: Keep Power Query when saving XLSX
-
core (PR1): https://gerrit.libreoffice.org/c/core/+/164290 (Merged)
-
core (PR2): https://gerrit.libreoffice.org/c/core/+/170691 (WIP)
-
submitted patchset 6:
- import&export “descr” and “title” attributes of element
-
next: use GrabBag for attributes - if possible
-
-
-
Skyler
-
Tiles rendering white when you enter edit mode (or zoom to certain levels)
-
Fixed and merged!
-
Hopefully this should allow us to get an Android release out to you shortly…
-
-
Fixed missing separation on invert background setting…
-
Lots of changes in this one… most of them are relation chain on Gerrit
-
Gerrit
-
GitHub
-
-
…also added save/load function so you keep in exactly the same theme when you reload a document/go to a new one…
-
…but calc does not work with saving/loading it properly yet, so there are some issues around refreshing your browser there. Hoping it’ll be a relatively simple fix…
-
-
Looking in to some jumping when scrolling horizontally on mobile…
-
Found another issue due to panning while I was looking at that, PR ready for review here:
-
Away for the first half of next week…
-
Elliot
-
Cell header context menu stayed open while interacting with the spreadsheet toolbar (allowing to switch sheets and the context menu would not close)
-
Fixed and merged
-
-
Changing what networks stats, CPU usage and and latency stats are recorded
-
feature added and merged
-
-
Refactoring Performance Measuring so that it will be in a single summary CSV
-
Will also include Git commit hash and date of commit
-
Will be in a format so a pivot table can be used to aggregate the data and create some nice charts
-
-
Will be away from the 7th-12th
-
-
Pranam
-
Gökay
Continues:
-
Separating global.js file from HTML.
- A big commit is merged. Thanks to Pedro, Skyler, Aron and Darshan for fixing some regressions while I was on vacation.
-
Helpers for object alignments.
Next:
-
Hunting for leaflet bits
-
Some small fixes and reviews.
-
Copy text with comments issue. Has a PR but needs to check the tests also.
-
Hubert
-
No updates
-
Gulsah
-
Changing dark mode affects other users link colors.
-
Ready to review
-
core patch: https://gerrit.libreoffice.org/c/core/+/169984 → needs reviewer while Szymon is out
-
online patch is not needed any more
-
-
-
Calc: Other users automatic font color affecting after one user inverts document background.
- Paused to work.
-
Add Invert background button to mobile view
-
Writer: Invert Background does not invert comments (GH#9629)
- Currently working on this.
-
Next: https://github.com/CollaboraOnline/online/pull/6725
- I’ll re-check that feature
-
Vivek
- No updates to share from my side, Thanks!
-
Timur
-
Just observing.
-
Méven
- Working on symbol insertion dialog regression caused by https://github.com/CollaboraOnline/online/pull/8011
-
Cor
-
Found the (easy) cause of a single keyboard shortcut issue/regression on Desktop, caused by a COOL commit.
Gökay will patch that. -
Dived into presentations/templates (weeks ago) trying to understand the cascading of styles and DF, which is ‘more interesting’ than e.g. Writer. Helps to clean up stuff. Later will look into my macro’s (from previous work) for cleaning/standardizing existing slides.
-
-
Nick
-
Working on moving the navigator to its own sidebar on the left
-
Converted current Control.Sidebar.js to typescript
-
Also working on making the sidebar more abstract so that we can reuse for the right and left side
- However this is more work than initially expected as the we listens to any sidebar events causing the sidebars to just mirror eachother
-
Next steps:
-
Figure out a way to decipher left and right sidebar events from core (no ‘left sidebar’ control exists in core).
- The slide sorter in core is closable so perhaps this might be some solution
-
Impress: add tabs for toggling between navigator and slide sorter as they are both on the left
-
Add a floating button to open the left sidebar when closed (similar button to when comments are closed)
-
-
Contextmenu key
-
Is enabled in core
-
Shouldn’t be much work to enable in online
-
-
GitHub issues → please add version within the body as the issue template states
-
Easy hack:
- Add Prettier fail check on pre-commit #9601 => assigned to pravo23 Pravin Kumar
- Save “Invert Background” preference in JS storage #9538
-
-
Top forum topics from last week:
-
Assistance Needed for Configuring Collabora Online with Nextcloud
-
Comment boxes are clipped when display scaling is 200% or higher → Anna / Ezinne please check this
-
Collabora Online Development Edition Spreadsheets:- Updated user about Note PDF Options are available in Collabora Online/CODE since some time
-
Docker Compose Nextcloud / Traefik / Collabora. WOPI Unauthorized, but Allow list empty
-
Failed to install Collabora-online on my own server
**
Python:( User: Gulixiang) => Thanks miklos for handling this topics**
-
In Impress, how do I use Python to insert text content at the cursor position?
-
How can I add a unique identifier to a table in a Word document?
-
How to obtain the currently selected image.Through Python scripting
-
[How does the server proactively send UNO commands to the client](http://How does the server proactively send UNO commands to the client)
-
Questions:
-
Karthik
I am happy to share that I completed building Mobile project for iOS and able to run in my device successfully. My requirement is to convert office files like docx, xlsx and ppt to PDF files completely offline in my current project. I tested the same feature in Mobile project from cool and it is working like a pro.
Could you please suggest how exactly to link this Mobile project to my current project to have that functionality in my project.
-
AI Darshan to drop here the SDK and simple example repository files with some text
-
IOS build info: https://collaboraonline.github.io/post/build-code-ios/
-
repository file:https://github.com/collaboraonline/online/tree/master/ios
-
-
- 08/01/24