Collabora Online Weekly Meeting #224

Development
1

Date: 03/07/2025

All minutes published on: https://forum.collaboraonline.com/tag/meeting-minutes

Attendees (write your name):

  • Ezinne, Skyler, Darshan, Pedro, Banobe, Méven, Gulsah, Parth, João, Attila, , Hub, Szymon, Luna Jernberg, Samuel Richard Anna

Testing

Files – ( password is coolmeeting)

Staging-perf

Share

  • Internal Link:

  • Features, user flows (Anyone can add here the things they need to test)

    • [Following Feature (mostly following the editor)] [the person who asked]: what to test
  1. Release Schedule

  • 25.04.3.2 to be released this week.

    • Issues with multiple users → bisecting is hard, not steady reproducible

    • Calc: offset with multiple users

      • Calc: Text (“blinking”) cursor

        • The text gets offset

      • Calc: Zoom in/out

      • Writer: not able to reproduce the inexistent text

  • Mobile release: submitting to Apple/Google for approval today, expected release tomorrow

  • 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 4

  • Salim

  • Sarper

    • Working local setup + Just listening in

  • Samuel

    • Build almost works

  • Richard B.

    • Just watching

  • Nicolas

  • Gabriel

  • Julius

  • Thorsten

    • [No update]

  • Banobe

    • Calc – dropdown menus offset with Zoom (In Progress)

      - So far fixed list dropdown positon, used html object position of dropdown

      - Now still trying to use coordinates of the autofilter dropdown to position Autofilter popup, still left with a few calculations to position the pop up on the right cell/column horizontally.

  • Attila

  • Extended Chart Styles: (Support font style themes, with customizable text element assignment).
    Implemented a popupwindow, and put iconview on it … still not implemented its behaviour and there are a lot of unused things to remove.
    I was able to render the chart for the iconView, I still fighting with some issues about it but probably it will work.
    I plan to make buttons for add/remove/save/load chart theme, next to the iconView.

  • Andras

  • Szymon

  • Stalled: Worked on PPTX export bug, added unit test, minimal reproducer and notes

    • new pointer from Tomaz, might be improt bug and confused shapes

  • Reviewing and adding pointers to UI rework issues

  • Updated old style previews in sidebar

    • merged core part

    • TODO: HiDPI and online part

  • Starting working on folding the notebookbar into dropdowns

    • extending the PR from community contributor

  • Darshan

    • PR:

      • Browser settings UI : https://github.com/CollaboraOnline/online/pull/12200

        • Thanks pedro for the svg

      • Calc: Layout tab: Add missing page group #12212 https://github.com/CollaboraOnline/online/issues/12212

      • C++ Easy hacks: https://github.com/CollaboraOnline/online/issues?q=is%3Aissue%20state%3Aopen%20label%3AC%2B%2B%20label%3A%22Easy%20Hack%22

        • Got few ideas from Ash on adding more c++ easy hacks → now attempting to convert them into easy hacks

          • Remove unused includes.

          • Sort includes and separate (with blank lines) into groups where the top group is internal headers, followed by Poco and finally C and C++ headers.

          • Replace, where possible, #ifdef MOBILEAPP with Util::isMobileApp() without re-writing code. In many cases this isn’t possible; leave those alone.

          • Find an eliminate unused utility functions (if really unnecessary).

          • Find duplicate/similar functions and propose a way to unify them. Note: Some functions are best left separate for performance.

          • Replace string+ concatenation in logs with operator<<.

          • Find opportunities to use constexpr.

          • Find opportunities to use const (but careful not to disable move semantics).

          • Remove const where std::move is used to promote moving.

          • Split the largest functions into more reasonable-sized ones. Note: Avoid splitting if it will force re-writing the function of duplicating significant code.

          • Split-out classes and independent code from large source files into their own files.

  • Aron

    • nothing to add

  • Caolan

  • Vivek

    • Impress Perf Improvements

    • fallback to 2D canvas when WebGL2 performance is insufficient

      • https://github.com/CollaboraOnline/online/pull/12162

      • Idea: Should we show warning dialogs/model for one time if we move them to 2d canvas?

        • Possible Problem: 1) Browser Accelerator off 2) Hardware Accelerator Off 3) Hardware is not compatible – Can’t do much :slight_smile:

          • Joao: could be useful. So, user can understand that it’s not a problem within COOL but actually something else (hardware; web browser; etc). The hard part is how to do that/

          • Banobe: Yes, I agree, but I don’t see how to do this with a common popup that automatically disappears

            • A warning with an ok button that allows the user to read the message and just after pressing the button → the message disappears

          • Pedro & Vivek:

            • I would rather not have an obtrusive message ….

            • we can do one more thing:

              • Add the message as a label directly in there

              • disable the ui in the edit mode for both the animations and transitions

                • Possible problem: people cannot change these things and save the document

  • Bayram

  • (In progress) – Writer: Title letters cut from below

    • investigating the issue…

  • Skyler

  • Bookmark dialog conversion to jsdialog

  • Android ANR/Closing issues

    • Looked at this some more, still don’t know much more than that we continue polling forever even when we have asked kit to shutdown

  • Pranam

  • Improving interoperability to import formulas from MSO

  • Fixing some async dialogues freezing new users joining

  • Gökay

Continues:

- Impress: Create Reset slide layout button · Issue #12168

- https://github.com/CollaboraOnline/online/issues/12168

- created an icon & mapped it to the AssignLayout uno command.

- https://github.com/CollaboraOnline/online/pull/12216

- SnackBar: Center align the OK text on snackbar button

- https://github.com/CollaboraOnline/online/issues/12174

- very simple easy hack, basically css modifications

- “do it for all the buttons” → will do it today

- Style preview: Add button that expands container · Issue #12171

- https://github.com/CollaboraOnline/online/issues/12171

- this is about expanding the writer’s styles preview to a popup, like how microsoft does

- i worked on aligning the buttons nicely to the center, removed the scroll buttons

- the popup is showing up with undefined entries (because more work is required)

- currently working on putting the popup on top of the styles preview

- i don’t know any css i realized =)

Moritz

  • Just listening

Samuel

  • Quick look at Gokay’s issue → related writer field, asked Michael S.

  • Looking at local build setup

    • problems with dependencies, namely poco version

    • Will probably send PR to improve build instructions

  • Online related work

    • Looking at an easy hack , good first issue. Maybe: Adding the accelerator to the toolbar

Luna * Just listening in before lunch

Events

Forum Topics

Announcement:

Next meeting

  • 10/07/2025