Skip to content

Make HTML renderers overridable#82

Closed
SjaakSchilperoort wants to merge 2 commits intodevelopfrom
feature/make-html-renderers-overridable
Closed

Make HTML renderers overridable#82
SjaakSchilperoort wants to merge 2 commits intodevelopfrom
feature/make-html-renderers-overridable

Conversation

@SjaakSchilperoort
Copy link
Member

No description provided.

@SjaakSchilperoort
Copy link
Member Author

@barry-observation Ik ben bezig met netjes doen van HTML-rendering. Dat blijkt een beetje een moeras te zijn. Deze PR beoogt om beter om te gaan met het overriden van renders voor HTML-tags, zodat we niet steeds een complete set renderers hoeven door te geven. Maar er blijkt meer aan de hand. De component gebruikt op verschillende plekken zaken uit theme en textStyle. Die zijn voor ObsIdentify en Observation niet gelijk, wat ervoor zorgt dat de rendering op Observation niet klopt.

Mijn voorstel is om de RenderHtml component aan zowel ObsIdentify als Observation toe te voegen, zodat dit meegaat in de theming van de apps, en dan uit react-native-components te verwijderen. Akkoord?

@SjaakSchilperoort
Copy link
Member Author

Deze wijziging gaan we doen na verhuizing van de component, in de app repo's. Op termijn komt dan een PR om RenderHtml te verwijderen uit deze repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant