HomePhabricator
Diffusion Comm 35a0538473a4

[web] Move `onChangeColor` up from `ThreadSettingsModal` to…

Description

[web] Move onChangeColor up from ThreadSettingsModal to ConnectedThreadSettingsModal

Summary:
Move onChangeColor out from inner class component to outer "Connected" functional component.

As part of the work to refactor ThreadSettingsModal... specifically to turn it into a functional component.


Depends on D3766

Test Plan:

  1. Open ThreadSettingsModal
  2. Change the color a bunch
  3. Log values in onChangeColor to ensure that they continue to be as expected
  4. Hit save after changing the color and ensure that the thread description changed as expected

Reviewers: def-au1t, palys-swm

Reviewed By: palys-swm

Subscribers: ashoat, Adrian, karol-bisztyga, benschac, yayabosh

Differential Revision: https://phabricator.ashoat.com/D3767