Skip to content

I228#15

Draft
pietrop wants to merge 6 commits intopietrop:performance-no-entities-spikefrom
bbc:i228
Draft

I228#15
pietrop wants to merge 6 commits intopietrop:performance-no-entities-spikefrom
bbc:i228

Conversation

@pietrop
Copy link
Owner

@pietrop pietrop commented Apr 23, 2020

Is your Pull Request request related to another issue in this repository ?

Describe what the PR does

viewing diffs

State whether the PR is ready for review or whether it needs extra work

NA
Additional context

NA

Pietro and others added 6 commits October 9, 2019 15:21
* fixes issue #133

* Working

if isEditable is true can edit speaker label, if false, cannot

* adjusted CSS accordingly
* fixes issue #133

* Working

if isEditable is true can edit speaker label, if false, cannot

* adjusted CSS accordingly

* moved CustomEditor in separate file

* speaker prompt keeps current speaker name

* brought in changes from previous PRs

master...philmcmahon:update-all-speakers#diff-cdf6a9957d22ff2efacaaf5fb2876a8aR58 it works but does not seem to rerender after change

* names change but component doesn't re-render

* changes don't propagate straight away but they do after you edit text

* cleaned up still no auto re-rendering

* works

added shoudl component update and component did mount in wrapper block to update speaker label

* changing spekaer triggers auto save callback

* Removed space

* Update packages/components/timed-text-editor/CustomEditor.js

Co-Authored-By: Eimi Okuno <emettely@users.noreply.github.com>

* Update packages/components/timed-text-editor/WrapperBlock.js

Co-Authored-By: Eimi Okuno <emettely@users.noreply.github.com>
* changed james to be point of contact for QA issues

* updated player controls

* updated images
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.

3 participants