-
Notifications
You must be signed in to change notification settings - Fork 380
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix #1367: Image picker enhancements #1805
base: dev
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the first contribution to this project.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR @Ateina!
I've added few things to update. Let me know if something's not right from your side 🙂
} | ||
|
||
.filePickerFolderCardTile { | ||
max-width: 176px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From what I tested, this property is not mandatory, as it's already calculated from the whole div.
background-color: "[theme:neutralLighterAlt, default:#{$ms-color-neutralLighterAlt}]"; | ||
.filePickerFolderCardImage { | ||
position: relative; | ||
width: 112px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For width values, I suggest to indicate viewport unit in order to be more responsive:
width: 112px; | |
width: 7vw; |
src/controls/filePicker/controls/DocumentLibraryBrowser/DocumentLibraryBrowser.module.scss
Outdated
Show resolved
Hide resolved
…ntLibraryBrowser.module.scss Co-authored-by: Michaël Maillot <[email protected]>
@michaelmaillot Thank you for review, I'm working on update for more responsive layout :) |
What's in this Pull Request?
PR contains changes to the FilePicker control to match OOTB Sharepoint File Picker
Before:
PR: