v0.5.0: Undo/Redo, New Space Header, Viewing/Editing modes #453
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* master: Addressed comments from PR Added background color for copied region Replacement should happen on defaulted value, not unsafe ref. Using 1/4em unicode spaces so individuals can format names as they like. webpack-dev-server should be exactly at 1.14.0 It shouldn't be possible for a non-hovered metric card to be in the editing state. fixed node modules.
* master: (27 commits) Added a comment to indicate some dead code. Renaming SimpleEditors as part of CR Basic changes for code review Add additional mac command key Stop breakage when empty cell is deleted Use NavHelper to ensure that backspace when nothing is selected will not go back in history MetricCardViewSection should always have relevant css attributes Force render of flowgrid on metric name change Cleaned text input css and made metric name not enterable when cannot be Catch event on undo Ensure that metric names update as is appropriate Refactor handleMouseDown and ensure that empty metrics are selected first click Fixed hasErrors mistype Only push updates of SimpleEditors when they are different than previous states Only render guesstimateForm if it is the correct metric Fixed a problem affecting dev where immutable invariants made space.organization a non-empty default object. Style TextInput correctly Removing broken npm deploy rules. TextForm editor should insertAtCaret with draftjs Fixing the persist edit state for the metric card name. ...
…orms Auto select metric text forms
…description within the metric modal.
* master: Addressed comments form cr Pressing escape on name field should deselect Pressing enter on empty cell should not create new line Auto select metric card name and then value Css fixes to allow editing in Safari Remove undefined 'event' call in TextInput. Fixes firefox, safari. Eliminating unnecessary errors and using Signup for sign up. Disalow newlines in text value editor
…sims Undo-redo should not resimulate needlessly.
Just edit and store the guesstimates natively; don't use the guesstimate form layer.
Add ref to connect to focus on distribution editor
* master: Added stage environments for master, ozzie, matthew Addressed comments from cr Minor color alterations Added cut functionality to space header Basic refactoring for header Basic toolbar for space show header Adjusted model header to experiment Bit more clean up. Clean Up Don't try to update the graph at all, just tell the user to refresh. Partially updates the graph. Space warning, not properly updating graph. We shouldn't update an undefined name to a '' name.
Probably needs a refactor later, but stops us from saving every key stroke.
Don't send previous updated at to disable the conflict check.
View options needs to reset both on mount and update
Only show the progress message in edits allowed mode
Space header bugs
Initialize checkpoints upon new model creation.
Only perform blur when text input has focus
… change, just the id does, (but we need the editableByMe prop check for when you load a space fresh, as there is no update where the id changes).
Making a new model should reset you to edit mode if you start in view mode.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.