a job well done

there are small things which you can do well which make a difference, things like weekly status reports. one way of doing it is just pasting the links quickly and adding one word status updates like "done" or "merged". this works, many do that and i used to do that till today but i realized that i was doing it just as a formality and no one will read them like that.

so i made up a structure of my own inspired from yaml. take a look at this week's report, it's nice on eyes, you don't have to open any links to understand what's going on. as you can see i prefer all lower case.

hi

development

- title: improve comments discoverability by adding option to show them on the navigator
- status: merged
- core: https://gerrit.libreoffice.org/c/core/+/189057
- github: https://github.com/CollaboraOnline/online/pull/12519
- comments: |
      i at first thought that `data.id` of a comment can be reliably considered it's
      position on the navigator and it worked just fine for older comments, but for
      any new comment, the ID was not the same thing as the comment's position relative
      to other comments, it was the highest, probably assigned from some counter. so i
      changed the uno command to pass strings around but szymon didn't buy that 🙂. i looked
      around a bit more in the code and found the constructs neccessary to get the comment
      from the ID. => learnt to always look around the code one is editing.

- title: style preview: add button that expands container (gh#12171)
- status: in progress (almost there)
- comments: |
      i found that the iconview constructor is being called three times, some debugging
      revealed that the first two times an empty iconview is created just as a placeholder,
      and the third time that placeholder is replaced with a populated iconview, thanks pedro 🙂.
      there's a lot more to this investigation, i will write about it in the next week's
      report with the status as merged hopefully.

regards,
sahil gautam