Skip to content

Conversation

@dataf3l
Copy link

@dataf3l dataf3l commented Nov 6, 2020

No description provided.

@sseerrggii
Copy link
Contributor

Thanks @dataf3l

This PR comes from our conversation, now the report looks like

Screenshot_2020-11-06 INFORME

Banco de Tiempo de Bogotà requests to add ID in order to identify the Post, I think is an easy and good improvement!

We will also have to think about improving reports feature in the future to made it more flexible

@sseerrggii sseerrggii requested a review from markets November 6, 2020 10:18
Copy link
Collaborator

@markets markets left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi 👋🏼 just a minor detail... would be preferable to avoid inline styles, maybe there is any Bootstrap (v3) helper to accomplish the same (or similar) visualization, IMHO it's worth to take a look.

Thanks!

@dataf3l
Copy link
Author

dataf3l commented Nov 6, 2020

Ok, I’ll take a look

@markets
Copy link
Collaborator

markets commented Nov 6, 2020

💡 maybe col-md-1 (or any col-md-*) would do the trick

PS since now we have 3 td, we should add a new th too

@markets
Copy link
Collaborator

markets commented Jan 7, 2021

Moved to 👉🏼 #581

@markets markets closed this Jan 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants