Harrison Healey

Results 170 comments of Harrison Healey

@M-ZubairAhmed Could you take another look at this to see if your feedback has been addressed as well?

Removing my review request on this to clear out some notifications. Let me know when this is ready for my review again

The web app now lives in the `webapp/channels` over in https://github.com/mattermost/mattermost-server. We decided to move it to be a monorepo to make both development and our CI pipelines easier, but...

Thanks for refactoring that. I really like the way this looks now, although I do think we should re-add the `/#landing` in the URL for both types of notifications. If...

Thanks for addressing my feedback. This looks good to me pending Alejandro's feedback then. Also, if you update the branch to be based off the latest master, that should fix...

I updated to the latest master again since we just had https://github.com/mattermost/mattermost/pull/26121 get merged which touched this same area of the code. I'm hoping I merged that correctly

@hossain-sazzad Which browser are you seeing that on? I'm on Mac as well, and Chrome/Safari/Edge/MM Desktop all scroll the right side when I press page up/down with the "Reply to...