6 May 2018 |
charlag | Sorry, I did not receive notification | 18:21:58 |
charlag | Passing null would be okay too | 18:22:09 |
charlag | Overload may work but I don't see much gain from it | 18:22:40 |
@remix2000:matrix.org | @charlag:matrix.org: Could you take a look at this commit to make sure that it is what we are talking about? :) | 18:46:10 |
@remix2000:matrix.org | I'm still not sure how to solve that in AccountActivity though :( | 18:47:04 |
charlag | Yeah, that's pretty much what I meant, cool. | 18:49:11 |
charlag | What's the problem with AccountActivity? You still need to show the picker? | 18:49:26 |
@remix2000:matrix.org | The problem is that there is a lot of things we need to reapply there, because who knows, maybe someone wants to follow themeselves from their other account? :P | 18:53:20 |
charlag | Why not lol? | 18:53:44 |
@remix2000:matrix.org | I mean, reapply after temporarily switching account. And also we should show which account is on the account page. Or maybe just finish AccountActivity after authorizing follow? | 18:55:09 |
@remix2000:matrix.org |
- which account is active on AccountActivity, I don't know how to say that properly :P
| 18:56:15 |
charlag | Finishing is fine but why not do the same override as in compose? | 18:56:48 |
charlag | This temprorary switching is just painful to deal with and probably confusing | 18:57:09 |
@remix2000:matrix.org | Follow button is most problematic there. | 18:57:50 |