Page MenuHomePhabricator

[native] fix rendering of edit thread name settings button
ClosedPublic

Authored by ginsu on Apr 24 2023, 11:16 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 4, 11:44 PM
Unknown Object (File)
Wed, May 1, 4:57 PM
Unknown Object (File)
Mon, Apr 15, 1:54 AM
Unknown Object (File)
Tue, Apr 9, 2:43 AM
Unknown Object (File)
Tue, Apr 9, 2:43 AM
Unknown Object (File)
Apr 3 2024, 5:54 PM
Unknown Object (File)
Mar 8 2024, 11:59 PM
Unknown Object (File)
Mar 7 2024, 7:16 AM
Subscribers

Details

Summary

As I was testing the loaders in Thread Settings for ENG-3741 I came across an issue where if the user navigated away from thread settings and back to it, then the button would not render while the new name was being saved. This was happening because nameEditValue was not being set since we navigated away from the screen so we were only just rendering SaveSettingButton which in turn render nothing since canChangeSettings is set to false while the name is being set. To fix this I created a separate function that just handles the rendering of the button and used that function within renderContent and now an ActivityIndicator will always render while the setting is being saved

A follow up diff for fixing the description will be up shortly

Test Plan

Please see the demo videos below to see the changes I made:

Before:

After:

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 24 2023, 12:35 PM