-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Probably shouldn't have collapse/expand widget ("twisty") for unpopulated Channels
and Publishers
views
#35157
Comments
I think I'm going to adopt "twisty" as the name for these from now on 😆 @martin-c0des should we just have some text there that says "no {publishers|channels}"? |
Hey, so I think we're going to revert this fix - talked to @LorenzoMinto about it and he wants to still display them. |
@fallaciousreasoning if we go that route, we should open a new issue instead of just reopening this one (things get messy when we reopen issues where code has landed). Thanks! |
Removing the However, it sounds like the above change needs to be reverted as per #35157 (comment). Going to label this as @fallaciousreasoning should the above even be verified/checked? Or should we just label is as |
Maybe let's just mark it as QA/No for now 😄 |
Description
Probably shouldn't have collapse/expand widget ("twisty") for unpopulated
Channels
andPublishers
viewsSteps to Reproduce
1.62.131
Turn on Brave News
Channels
orPublishers
-- leave it without subscribed sourcesActual result:
Nothing happens, nothing's there - the widget just moves
Expected result:
I'd expect it to not-yet have the "twisty" collapse/expand widget, until we have source content
Reproduces how often:
100%
Brave version (brave://version info)
Version/Channel Information:
Other Additional Information:
Miscellaneous Information:
cc @martin-c0des @aguscruiz @fallaciousreasoning @mattmcalister @rebron @brave/qa-team
The text was updated successfully, but these errors were encountered: