react-spectrum icon indicating copy to clipboard operation
react-spectrum copied to clipboard

CardView docs

Open LFDanLu opened this issue 3 years ago โ€ข 15 comments

Closes #2729

โœ… Pull Request Checklist:

  • [ ] Included link to corresponding React Spectrum GitHub Issue.
  • [ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • [ ] Filled out test instructions.
  • [ ] Updated documentation (if it already exists for this component).
  • [ ] Looked at the Accessibility Practices for this feature - Aria Practices

๐Ÿ“ Test Instructions:

๐Ÿงข Your Project:

RSP

LFDanLu avatar Jan 18 '22 19:01 LFDanLu

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 18 '22 19:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 18 '22 20:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 19 '22 00:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 19 '22 00:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 19 '22 00:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 20 '22 23:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 21 '22 01:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 21 '22 20:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 21 '22 22:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 21 '22 22:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 21 '22 22:01 adobe-bot

Build successful! ๐ŸŽ‰

adobe-bot avatar Jan 22 '22 01:01 adobe-bot

@reidbarber thats weird, are you able to reproduce it consistently? I saw this happen a long while ago I think but couldn't reproduce it, I can take another look

LFDanLu avatar Feb 01 '22 18:02 LFDanLu

@reidbarber thats weird, are you able to reproduce it consistently? I saw this happen a long while ago I think but couldn't reproduce it, I can take another look

Interestingly, I'm getting a mix of the first and second image's layout when I refresh, and can't seem to find a pattern. My guess is that it's some race condition with the order that the images are loaded. Still weird that it would change when a selection is made though. This was at 1920x1080 btw. Also tried a mix of refreshing and hard refreshing.

reidbarber avatar Feb 01 '22 18:02 reidbarber

@reidbarber From what I can tell, https://github.com/adobe/react-spectrum/blob/main/packages/@react-aria/virtualizer/src/useVirtualizerItem.ts#L39-L46 is returning equal heights for each card div when this bug happens. Still poking around to see if this is something related to the images loading or something else

LFDanLu avatar Feb 02 '22 00:02 LFDanLu