zotero-libreoffice-integration icon indicating copy to clipboard operation
zotero-libreoffice-integration copied to clipboard

Icons

Open bijang opened this issue 6 years ago • 22 comments

Zotero icons need a design overall ideally, to fit hidpi, due to most common icons used by v6.3, to be compatible with the new toolbar/groupbar variations available from v6.2 onwards, and possibly even different designs to fit the most common styles (breeze, colibre, elementary)

bijang avatar Jan 06 '19 12:01 bijang

Currently I am the maintainer of official icon theme in LibreOffice, I might give a hand to this request.

rizmut avatar Sep 25 '21 04:09 rizmut

Helo, can I work on this request?

Just curious, where the icon definition is placed? The icons itself are placed under build/oxt/icon. I would like to update them to 24px size instead of 26px

rizmut avatar Mar 16 '22 05:03 rizmut

Helo, can I work on this request?

Do you know how to bundle different icons for different themes in an extension? How does libreoffice icon theming work?

Just curious, where the icon definition is placed?

https://github.com/zotero/zotero-libreoffice-integration/blob/master/build/oxt/Addons_AOO4.xcu#L107-L186

The icons itself are placed under build/oxt/icon. I would like to update them to 24px size instead of 26px

The icons are 26x26px because that's the requirement. I cannot find any official LibreOffice docs on this on the current web and LibreOffice docs have always been nigh-impossible to locate, but here's one place that says so. It doesn't make much sense to me either if you ask me, but that's what the docs said when we last updated the icons ourselves.

adomasven avatar Mar 21 '22 11:03 adomasven

Will the new “Add Note” button be added at some point also? I don't have MS Word, but I presume it is already on that platform, Thank you

bijang avatar Mar 22 '22 04:03 bijang

@bijang: Add Note is available on all platforms. If you're not seeing it in LO, reinstall the plugin from the Cite preferences. Post to the forums if you're still having trouble.

dstillman avatar Mar 22 '22 04:03 dstillman

thank you that's working fine

bijang avatar Mar 22 '22 04:03 bijang

Do you know how to bundle different icons for different themes in an extension?

As far as I know, there's no support for extensions to bring different themes currently from within LibreOffice theming mechanism. Another option is extension makers who provide the option to change the theme, but this means adding a new workload to them

How does libreoffice icon theming work?

I have zero understanding about code, so here I can not explain much. But I can bring anyone who interested against the topic to the right person.

The icons are 26x26px because that's the requirement. I cannot find any official LibreOffice docs on this on the current web and LibreOffice docs have always been nigh-impossible to locate, but here's one place that says so. It doesn't make much sense to me either if you ask me, but that's what the docs said when we last updated the icons ourselves.

26px x26px size was "official" icon theme called Galaxy from Sun Microsystem era, the LibreOffice team currently agrees to use 24px as it's standard size in FreeDesktop. But yea, so unfortunate no official docs here.

rizmut avatar Apr 20 '22 11:04 rizmut

As far as I know, there's no support for extensions to bring different themes currently from within LibreOffice theming mechanism.

That was my understanding too.

Another option is extension makers who provide the option to change the theme, but this means adding a new workload to them

I'm afraid we're not interested in it if it requires separate releases.

adomasven avatar Apr 20 '22 12:04 adomasven

The icons are 26x26px because that's the requirement

It was in the Galaxy days, but we have dropped that many years ago, and I changed the Large icons to 24 instead in https://github.com/libreoffice/core/commit/bea44e5c1b982d05314b72452b94fe288071a496

fitojb avatar Aug 19 '22 12:08 fitojb

@fitojb Are there any up-to-date docs for LibreOffice available somewhere that document modern plugin development?

adomasven avatar Aug 29 '22 12:08 adomasven

@adomasven Have you looked at https://wiki.documentfoundation.org/Development/Extension_Development ? If you’re missing something in particular, please make note of it on the talk page or send questions to the mailing list [email protected]

fitojb avatar Aug 30 '22 13:08 fitojb

Currently I am the maintainer of official icon theme in LibreOffice, I might give a hand to this request.

@rizmut if you can work on it, I will PR here to add them :)

raffaem avatar Oct 08 '22 07:10 raffaem

Oh yes the work is on progress

@rizmut if you can work on it, I will PR here to add them :)

rizmut avatar Nov 05 '22 02:11 rizmut

zotero-colibre.zip This is my work. For large size, I set 24px instead of 26px to match the latest convention changes. Actually, it still needs an extra large size of 32px. Please let me know if that size would like to be added too.

rizmut avatar Nov 05 '22 03:11 rizmut

I got lost.

Relevant files seem to be in ./build/oxt/icons.

They are not split by LibreOffice theme? So there is the same set of icons for all possible LibreOffice theme?

How plugin icons are supposed to be integrated with the main theme?

Shouldn't this be managed at the theme-level, instead of at the plugin-level?

raffaem avatar Nov 05 '22 08:11 raffaem

zotero-colibre.zip This is my work. For large size, I set 24px instead of 26px to match the latest convention changes. Actually, it still needs an extra large size of 32px. Please let me know if that size would like to be added too.

image

It looks really nice and integrates nicely with the overall colibre theme.

Although I probably would have colored the "Refresh" button in blue, but that's just taste 😁

raffaem avatar Nov 05 '22 08:11 raffaem

@rizmut I don't understand what setDocPrefs_24-a is for. It seems it's not defined in the .xcu files?

raffaem avatar Nov 05 '22 08:11 raffaem

@rizmut I don't understand what setDocPrefs_24-a is for. It seems it's not defined in the .xcu files?

It's just a variation to choose. Either you go with setDocPrefs_24-a or go with setDocPrefs_24

rizmut avatar Nov 09 '22 06:11 rizmut

@rizmut I don't understand what setDocPrefs_24-a is for. It seems it's not defined in the .xcu files?

It's just a variation to choose. Either you go with setDocPrefs_24-a or go with setDocPrefs

Ok, I went with the latter.

I PRed this as promised :D

raffaem avatar Nov 09 '22 07:11 raffaem

zotero-colibre.zip This is my work. For large size, I set 24px instead of 26px to match the latest convention changes. Actually, it still needs an extra large size of 32px. Please let me know if that size would like to be added too.

Hi,

upstream is not interested in my PR (see here).

I agree it's better to have the icons of the extensions change automatically when the user switches theme inside LibreOffice. But this seems not possible with LibreOffice, and I think that we have what we have, and we need to live along with it :)

So I published the extension with your icons bundled on my fork.

If you ever make new icons for new themes, just open an issue there.

It should be trivial to add new icons :)

raffaem avatar Nov 09 '22 09:11 raffaem

Ah, what a shame, these icons look amazing @rizmut. Maybe they will be upstreamed in the future 🙏

hydroEng avatar Nov 10 '22 23:11 hydroEng

Hi,

upstream is not interested in my PR (see here).

I agree it's better to have the icons of the extensions change automatically when the user switches theme inside LibreOffice. But this seems not possible with LibreOffice, and I think that we have what we have, and we need to live along with it :)

Just wondering why upstream reject all of the PR? I means at least incorporating one of icon theme will not be a huge problem for them.

But thank a bunch for your PR. I am really too excited to add more icon themes if proper support exists.

rizmut avatar Nov 11 '22 22:11 rizmut