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

WIP Support poll vote/unvote #939

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

PIG208
Copy link
Member

@PIG208 PIG208 commented Sep 12, 2024

This is stacked on top of #912.

Fixes: #166

This prepares for a later change that introduces new content types
like polls and todos.

Signed-off-by: Zixuan James Li <[email protected]>
The UI follows the webapp until we get a new design.

The dark theme colors were tentatively picked. The `TextStyle`s are
the same for both light and dark theme. All the styling are based on
values taken from the webapp.

References:

  - light theme:

    https://github.com/zulip/zulip/blob/2011e0df760cea52c31914e7b77d9b4e38e9ee74/web/styles/widgets.css#L138-L185
    https://github.com/zulip/zulip/blob/2011e0df760cea52c31914e7b77d9b4e38e9ee74/web/styles/dark_theme.css#L358

  - dark theme:

    https://github.com/zulip/zulip/blob/2011e0df760cea52c31914e7b77d9b4e38e9ee74/web/styles/dark_theme.css#L966-L987

Fixes zulip#165.

Signed-off-by: Zixuan James Li <[email protected]>
The route is not documented in the Zulip API Documentation.

Instead, we refer to the zulip-mobile implementation and
https://zulip.readthedocs.io/en/latest/subsystems/widgets.html#polls-todo-lists-and-games.

Signed-off-by: Zixuan James Li <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

polls: Vote and unvote on options
1 participant