Proposal: Adding `arrow-expand` and `arrow-compress` to the icon themes

Context

The gallery macro’s maximize/minimize button currently uses two hard-coded GIF images, which don’t fit a page with a white background and ignore the icon theme. None of the existing icon names describe an expand/collapse-to-fullscreen action. arrows could stand in for “expand”, but nothing matches “compress”.

Proposal

Add two icon mappings to the XWiki icon set:

  • arrow-expandmaximize: FontAwesome expand, Silk arrow_out
  • arrow-compressminimize: FontAwesome compress, Silk arrow_in


Opinion

+1 from me
You can see the full mapping in action at: XWIKI-22770: Gallery macro - adapt elements for a page that has white background by Sereza5 · Pull Request #4242 · xwiki/xwiki-platform · GitHub

Conclusion

Do you agree with adding this pair of names to the IconSet?
Are arrow-expand / arrow-compressmaximize/minimize good names?

If nobody objects within a week, I’ll consider the proposal accepted. I’ll then update PR #4242 and the icon documentation to match what we decide if it gets merged.

Thanks for reading, I’m looking forward to your feedback :slight_smile:
Lucas C.

EDIT: changed the proposed naming to go along with the discussion at Proposal: Adding `arrow-expand` and `arrow-compress` to the icon themes - #4 by CharpentierLucas

Hi,

+1 for me.
Note that I will also need these icons for XWIKI-24729: Make livedata's edit mode better integrated in XWiki's UI by pjeanjean · Pull Request #6335 · xwiki/xwiki-platform · GitHub, I was going for the names maximize and minimize but I’m fine either way.

+1 especially if we already have two distinct needs for the same icons.

arrow-compress and arrow-expand are very specific, I think maximize / minimize are more generic and reusable, I agree to change the naming for those :+1:

Thanks for the help :slight_smile:

@CharpentierLucas +1 but I’d like that we verify if there are icons for this in other icon sets (for ex Glyphicon (Bootstrap) and Elusive - see https://design.xwiki.org/xwiki/bin/view/Proposal/XWiki%20Icon%20Set ). That would ensure that the icon is common enough (finding just the mapping for 2 icon sets is a bit small for a good stat). I’m not too worried since that seems like a common icon, but I’d like that we follow our process.

See https://www.xwiki.org/xwiki/bin/view/Documentation/DevGuide/FrontendResources/Icons/#HUpdatingtheiconset and especially:

If there’s a need for a new icon, the XWiki icon set can be extended. Such an extension is a non reversible change, it should be discussed with the community on the forum. In the proposal, make sure to provide a mapping for at least four icon themes: Silk, FA4, FA5, Glyphicons and the Material Icons.

Thx!

+1

Here’s a mapping that’d work:

Icon theme maximize minimize
Glyphicons (Bootstrap 3) resize-full resize-small
Elusive 2 resize-full resize-small

+1, thank you!

Thank you for all the feedback!
Closing with 6 +1 and proceeding with their addition in https://github.com/xwiki/xwiki-platform/pull/4242
I’ll update the doc when this PR gets merged :slight_smile: