Neil Marcellini

Results 153 comments of Neil Marcellini

This was deployed 2 weeks ago, what needs to be done to get it closed out @isabelastisser?

> @sahilnagpal26's [proposal](https://github.com/Expensify/App/issues/35763#issuecomment-1925562218) correctly identifies the root cause and the suggested solution is valid. > > 🎀👀🎀 C+ reviewed. @sahilnagpal26 It seems like the solution probably works given that it's...

@sahilnagpal26 thank you for updating your proposal with a better root cause explanation. It makes sense to me. However, I don't think your explanation of the solution is correct, regarding...

The display name is bold in search regardless of the unread status of that chat, so I don't understand why that's part of the test steps. I think there is...

It would make more sense if the style of the chats on the search page matched the LHN. That's not the case on staging or prod. What do you think...

> I think for the chat switcher, it does make sense that the chat matches how it would look in the LHN. So if the chat is unread, keep it...

Ok so it sounds like this is the desired behavior. I'm going to update the issue description and send it to be implemented externally. - Chats on the search page...

Sorry for the delay, looking now