-
Notifications
You must be signed in to change notification settings - Fork 306
List of topics in channel #1158
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
Comments
Tim and others propose featuring this page more prominently than the interleaved channel view: Tim said:
|
And another long-time Zulip user said:
|
Another user said:
|
From a review in the Google Play Console:
I assume we'll want this list of topics to be searchable, like in zulip-mobile. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Add "Show topics" button tooltip and "Search topics" placeholder text to all language files (English, Slovak, Norwegian, Polish, Russian, Arabic, and Japanese). Fixes part of zulip#1158.
This commit creates a new topic list page showing all topics in current channel with search functionality ordered by recency Fixes part of zulip#1158.
This commit creates a new topic list page showing all topics in current channel with search functionality ordered by recency Fixes part of zulip#1158.
Add topic list button to app bar actions in message list view. The button appears in channel narrows and navigates to the topic list page when pressed. Uses the topic icon and localized tooltip. Fixes zulip#1158.
This commit creates a new topic list page showing all topics in current channel with search functionality ordered by recency Fixes part of zulip#1158.
Add topic list button to app bar actions in message list view. The button appears in channel narrows and navigates to the topic list page when pressed. Uses the topic icon and localized tooltip. Fixes zulip#1158.
This commit creates a new topic list page showing all topics in current channel with search functionality ordered by recency Also adds a button to app bar actions in message list page which appears in channel narrows and navigates to the topic list page when pressed Fixes zulip#1158.
This commit creates a new topic list page showing all topics in current channel with search functionality ordered by recency Also adds a button to app bar actions in message list page which appears in channel narrows and navigates to the topic list page when pressed Fixes zulip#1158.
@lakshya1goel plans to implement this feature; see #mobile-design > Topic list in channel @ 💬. |
We now have a design for this feature; I've updated the issue description with details. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hi, PR #1449 is ready for review, PTAL. Thanks! |
This came up in beta feedback: #mobile > Flutter Beta: Some Feedback @ 💬 |
Copying here what I wrote just now (#1449 (comment)) on the PR: Thanks @lakshya1goel for your work on this issue. I think we'll be able to get this to something we're happy to merge much more quickly by having one of us on the core team drive it, so I've asked @PIG208 to take it on. |
The topic-list page app bar should match the Figma design except for its height. That part needs to be coordinated with other pages. Also on the topic-list page app bar, the icon "chveron_down.svg" (originally "chveron-down-16.svg") comes from: https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=6820-36184&m=dev and not the "Icons" page, since the "chveron-down" there appears to be different from the one on the topic list page design. We also leave out the new topic button and topic filtering. Those are out-of-scope for zulip#1158. --- On the message-list page, "TOPICS" is not aligned to the middle part of the app bar, that's because we haven't got to zulip#1039 yet. --- The topic-list implementation is quite similar to parts of inbox page and message-list page. Therefore, we structure the code to make them look similar to compare for changes side-by-side to help with reviewing what has changed. Figma design: https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=6819-35869&m=dev
The icon was taken from CZO discussion: https://chat.zulip.org/#narrow/channel/530-mobile-design/topic/Topic.20list.20in.20channel/near/2140324 Fixes: zulip#1158 Co-authored-by: Zixuan James Li <[email protected]>
The icon was taken from CZO discussion: https://chat.zulip.org/#narrow/channel/530-mobile-design/topic/Topic.20list.20in.20channel/near/2140324 Fixes: zulip#1158 Co-authored-by: Zixuan James Li <[email protected]>
The topic-list page app bar should match the Figma design except for its height. That part needs to be coordinated with other pages. Also on the topic-list page app bar, the icon "chveron_down.svg" (originally "chveron-down-16.svg") comes from: https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=6820-36184&m=dev and not the "Icons" page, since the "chveron-down" there appears to be different from the one on the topic list page design. We also leave out the new topic button and topic filtering. Those are out-of-scope for zulip#1158. --- On the message-list page, "TOPICS" is not aligned to the middle part of the app bar, that's because we haven't got to zulip#1039 yet. --- The topic-list implementation is quite similar to parts of inbox page and message-list page. Therefore, we structure the code to make them look similar to compare for changes side-by-side to help with reviewing what has changed. Figma design: https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=6819-35869&m=dev
The icon was taken from CZO discussion: https://chat.zulip.org/#narrow/channel/530-mobile-design/topic/Topic.20list.20in.20channel/near/2140324 Fixes: zulip#1158 Co-authored-by: Zixuan James Li <[email protected]>
This removes the ad-hoc color we use for null subscriptions, which is a workaround for not having logic to generate different base colors within constraints for unsubscribed channels: https://chat.zulip.org/#narrow/channel/48-mobile/topic/All.20channels.20screen.20-.20.23F832/near/1904009 While this change does not implement that logic either, it removes the ad-hoc color in favor of a constant base color, intended for colorSwatchFor. The base color can be reused on topic-list page (zulip#1158), giving us access to `.iconOnBarBackground` and friends for the app bar.
In the legacy app, if you're looking at the messages in a topic, we have a button in the app bar (a "list" icon) that gets you a list of all the topics in that channel. It's sorted by recency, and shows the number of unreads, whether you have an @-mention, and whether the topic is muted etc.
Basically it's similar to what you get in the web left sidebar after hitting "Show all topics" under a given channel.
We should offer the same thing in the new app. I wasn't sure people really use that feature; but it came up in user feedback after yesterday's beta announcement, so that's a sign that people do.
Design
Design is in Figma:
https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=6819-35869&m=dev
For this issue, differences from the design there:
Implementation
The list of topics will come from making a request to the server, after opening the page.
As a follow-up task, after getting the list for a given channel we might store it (on
ChannelStore
), and keep it up to date when handling events, so that coming back a second time to this topic-list page for a given channel is fast. For this PR, though, that's out of scope. Instead the list of topics can just live on aStatefulWidget
for the page.References
The text was updated successfully, but these errors were encountered: