Collabora Online Weekly Meeting #223

Development
1

Date: 27/06/2025

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

Attendees (write your name):

  • Ezinne, Skyler, Darshan, Pedro, Banobe, Bayram, Méven, Gulsah, Parth, João, Attila, Sahil, Mohit, Moritz, Hub, Vivek, Samuel

Testing

Banobe (AI) : Create a GH issue on the snack bar OK button being left aligned instead of centred aligned → @Banobe Pascal is this done? Can you paste here the link pelase
https://github.com/CollaboraOnline/online/issues/12174

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.05.3.1 is released (today)

  • Now we are in the 2****nd week of green cycle, next week we will release CODE Green

    25.04.3.2

    • Issues with multiple users → Joao bisecting

    • 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: planning to release next week (week beginning 30th June 2025)

  • 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

  • Neil Mohr

    • Just watching

  • Nicolas

  • Gabriel

  • Julius

  • Thorsten

    • [No update]

  • Banobe

    • Fix menubutton alignment and styling inconsistencies (Merged)

    • RTL: Fix spinfield padding for rtl to properly center icons and values (Merged)

    • Fix Calc regression: sheet tabs not visible and cannot create new sheet
      - Later will need to add a cypress visual test to cover sheets toolbar

    • Calc – dropdown menus offset with Zoom (In Progress)
      - Tried a new implementation where the dropdowns start from the dropdown icon and with this regardless of zoom on browser or document the popup position will be properly aligned to the dropdown

  • Attila

  • Extended Chart Styles: (Support font style themes, with customizable text element assignment).
    Implementing UI.. try to follow stylePreview as an example as it use IconView, already made a sidebar panel, Still implementing the popup window for the dropdown list.
    I plan to make a popupmenu too with commands save/load/delete.

  • Andras

    • Change: repository signing keys

      • When you upgrade CODE or COOL you might get a warning, don’t be scared

      • Necessary because some distro started rejects those key types since they are already deprecated GPG key algorithm

      • So, you need to update / pull the new key

  • Szymon

  • Worked on PPTX export bug, added unit test, minimal reproducer and notes, weird thing during save - asked Tomaz to see

  • Added code pointers to UI issues

  • Updated old style previews in sidebar patch for next RED

  • Starting working on folding the notebookbar into dropdowns

  • Darshan

    • PR:

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

        • Ping Pedro on the SVGs topic

      • 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+ concatentation 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

  • Some work underway for memory optimizations

  • Working on a similar memory optimization to preload typedetection data

  • Auto committed Watchdog profiles

  • Auto committed copy-on-write/do_wp_pagege profiles

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

  • 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

  • (Closed) - tracked changes numbering (tdf#157596)

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

  • Skyler

  • Mobile release

    • Expecting to release 25.04.2 next week

  • Bookmark dialog conversion to jsdialog

    • Worked on editing text in treeview – it nearly works!

      • Core can now get updates on an edit

      • Still need to find if fields are editable before letting you edit them

  • 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 =)

Events

Forum Topics

Announcement:

Next meeting

  • 03/07/2025