-
Notifications
You must be signed in to change notification settings - Fork 7
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
Resources component #77
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
The resource tiles should always have a white background as they're essentially a card so the content can read better within the accordion. Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com> # Conflicts: # src/develop/Develop.js # src/examples/Examples.js # src/scss/_objects.modules.scss
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
description: 'Real Academia de Bellas Artes Ehances Experience with Mobile App' | ||
} | ||
]} | ||
/> | ||
</AccordionPanel> | ||
<AccordionPanel panelTitle="Software Licensings and Managment"> |
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.
Can you please fix the typo on this line? I tried fixing it but have no permission to write to your fork. Managment
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.
Fixed the typo, also added you as a collaborator to my fork
</dl> | ||
</section> | ||
|
||
<section> |
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.
Can we list out the ResourcesTile's options here in details, similar to how the Tags/Tag components documentation in Grommet core?
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com>
Signed-off-by: Emmanuel Pilande <manny.pilande@hpe.com> # Conflicts: # src/examples/Examples.js
# Conflicts: # src/develop/Develop.js # src/scss/_objects.callout.scss # src/scss/_objects.modules.scss
A general question, is there a case where a ResourceTile is a link? It looks to me like these ResourceTiles should be clickable elements. If so, it does not look like the ResourceTile component supports that functionality in this PR, right? |
# Conflicts: # src/develop/Develop.js # src/examples/Examples.js # src/scss/_objects.modules.scss
@phuson agreed. Updated PR to make resource tiles clickable |
Resource & Resource Tile component #32.
Also related to #20.
Screenshot of component:
![image](https://cloud.githubusercontent.com/assets/3210082/16067898/9b01bed8-325c-11e6-9e51-a683ceaf67b8.png)