question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[feat request] Gallery should add "maxCol" and "minCol"

See original GitHub issue

Describe the issue. What is the expected and unexpected behavior?

The current gallery layout is not able(or very difficult) to custom columns. In some wide screen, we might only want to 3 or 4 columns, but the default columns are 6-7. it makes every item only show limit width.

I suggest add maxCol and minCol props to Gallery.

Please provide the steps to reproduce. Feel free to link CodeSandbox or another tool.

Is this a bug or enhancement? If this issue is a bug, is this issue blocking you or is there a work-around?

What is your product and what release version are you targeting?

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Comments:5 (4 by maintainers)

github_iconTop GitHub Comments

1reaction
mcokercommented, Feb 4, 2021

Hi @kunyan we have an open issue to implement the CSS var you referenced, as well as a max width var - https://github.com/patternfly/patternfly-react/issues/5055. I imagine that will be implemented as min/max props on the <Gallery> component that set the CSS vars inline. Those vars also support being defined at specific breakpoints, and are documented here - https://www.patternfly.org/v4/layouts/gallery/html#documentation

Until support for defining the min/max vars via props on the react component is implemented, you can always set the variables in your own CSS for a particular gallery layout. Let me know if you need any help with that.

0reactions
mcokercommented, Jun 1, 2021

closed via https://github.com/patternfly/patternfly-react/pull/5735, implemented as minWidths and maxWidths on <Gallery>

Read more comments on GitHub >

github_iconTop Results From Across the Web

src/screen.c - third_party/vim - Git at Google
settings), must call redraw_later(NOT_VALID) to have the whole window ... match_T *shl, linenr_T lnum, colnr_T mincol, matchitem_T *cur);.
Read more >
Seurat source: R/visualization.R - Rdrr.io
R/visualization.R defines the following functions: Transform SingleSpatialPlot SingleRasterMap SinglePolyPlot SingleImagePlot SingleImageMap SingleExIPlot ...
Read more >
Harmony Enterprise™ 2022.3 Help – PDF
Harmony Enterprise is a comprehensive engineering application for ... AppStream 2.0 must first have a base image created with Harmony ...
Read more >
https://opensource.apple.com/source/vim/vim-44/src...
Commands that change everything (e.g., resizing the screen) must call ... last known cursor position */ #ifdef FEAT_SEARCH_EXTRA static match_T search_hl; ...
Read more >
Hybrid Segmentation Method With Confidence Region ...
tumor image. Those features must be reduced with accuracy. This issue and its importance of tumor identification can be.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found