Skip to content
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

Change matter/legislation description to a list of keywords/topics #168

Closed
tohuynh opened this issue Jan 10, 2022 · 3 comments · Fixed by #261
Closed

Change matter/legislation description to a list of keywords/topics #168

tohuynh opened this issue Jan 10, 2022 · 3 comments · Fixed by #261
Assignees
Labels
enhancement New feature or request

Comments

@tohuynh
Copy link
Collaborator

tohuynh commented Jan 10, 2022

Feature Description

In the event page's voting tab,
image
These legislation descriptions can be too long for a cell, we should use top 5 keywords or topics to describe the legislation instead.

Use Case

Please provide a use case to help us understand your request in context.

The list of keywords/topics is more concise and is better looking overall in the table than the long description

Solution

Please describe your ideal solution.
This isn't ready for dev as the matter index pipeline isn't ready

Can't be done until matter index pipeline is done but i do really like this idea!

Originally posted by @JacksonMaxfield in #155 (comment)

Alternatives

Please describe any alternatives you've considered, even if you've dismissed them.

Screenshots/References

In the case of a visually oriented feature or component request, providing drawings, screenshots, and/or links to examples of what you're seeking will be very helpful.

@tohuynh tohuynh added the enhancement New feature or request label Jan 10, 2022
@evamaxfield
Copy link
Member

As a temporary work-around we might just want to cut it off after some number of characters? 512 characters and then ...?

@tohuynh
Copy link
Collaborator Author

tohuynh commented Jan 10, 2022

Yes, that could work as a temporary fix.

@tohuynh tohuynh self-assigned this Jan 12, 2022
@tohuynh
Copy link
Collaborator Author

tohuynh commented Jan 13, 2022

Going to hold off on working on the temporary fix as the solution might be tied to #163. When the votes are expanded it might be worth it to show the entire legislation description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants