Commit e98ccf5
WIP: Comment glyphs (codesandbox#3726)
* initial stuff
* initial glyph implementation
* optimistic transposing
* add active class to glyphs
* WIP: add comment from context
* handling optimistic thread
* multi comments on line
* favor active over multi icon
* add numbers
* copy changes from codesandbox#3729
* remove trigger check
* remove react-draggable coz we have framer motino
* open file and scroll to position
* refactored most, continuing on laptop...
* fix replies
* glyph likes themes
* oops that not supposed to be there
* extract avatarblock
* move edit comment out
* use edit content in reply
* add dialog positioning
* always render even if position is not known
* better fallback position
* remove unused ref
* add polling mechanism to active comment
* change the fallback just a little more
* optimistically set position
* fix checking id in comment glyph boundary as well
* remove optimistic trigger
* improve moving comments
* use currentTarget instead
* set placeholder + animate from left in fallback
* move position logic into a funktion
* animate based on _file change_
* rename optimistic to newComment
* add permalinks (codesandbox#3744)
* add permalinks
* remove query from effect
* move copy to clipboard to effect
* add defaults
* create action for commenturl
* yaay
* Apply suggestions from code review
Co-Authored-By: Christian Alfoni <[email protected]>
* Update packages/app/src/app/pages/Sandbox/index.tsx
Co-Authored-By: Christian Alfoni <[email protected]>
* make action
Co-authored-by: Christian Alfoni <[email protected]>
* fixed add comment position and decorate line
* redesign new comment
* fix ... icon
* pull out new comment component
* rename to AddComment duh
* move add reply to a component
* pull out comment header
* pull out comment body into a component
* close dialog is lame
* but i am lamer
* leave a comment before i forget
* add comment glyph
* some fixes
* window collisions - first draft
* cheeky scale guessing
* found the missing 48px :P
* dont need to track animation state now
* always div for unknown semantics
* fix comment current target trigger
* omg strings!
* fix animations on comments
* improve handling optimistic
* use animate controller
* add comments
* Add times and move file name (codesandbox#3752)
Co-authored-by: Christian Alfoni <[email protected]>
* fix issue with moving from optimistic to existing comment
* refactor to initial and end position
* ensure we actually have a comments state for the sandbox
* fix multi popup
* auto focus on creating new comment
* add permission checks on comments
* drag handle!!!
* improve dragging
* style multi comments (codesandbox#3763)
* style multi comments
* use element
* update lookup
* fix ot typing
* fixed types and defaulting to comments
* fix typing
* last fixes
Co-authored-by: Sara Vieira <[email protected]>
Co-authored-by: siddharthkp <[email protected]>1 parent 414f01b commit e98ccf5
File tree
49 files changed
+2516
-1389
lines changed- packages
- @types/ot
- app
- src/app
- components/CreateNewSandbox/CreateSandbox/Import
- graphql
- overmind
- effects
- gql/comments
- utils
- vscode
- namespaces
- comments
- editor
- files
- live
- utils
- pages/Sandbox/Editor
- Header/Collaborators
- Workspace
- screens
- CommentThreads
- Dialog
- Comments
- Dialog
- components
- static/img
- common/src/types
- components/src/components/ThemeProvider
- homepage/content/articles/redefining-code-collaboration
- standalone-packages/vscode-textmate
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
49 files changed
+2516
-1389
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
25 | 36 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
194 | 194 | | |
195 | 195 | | |
196 | 196 | | |
197 | | - | |
198 | 197 | | |
199 | 198 | | |
200 | 199 | | |
| |||
Lines changed: 2 additions & 16 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
31 | 31 | | |
32 | 32 | | |
33 | 33 | | |
34 | | - | |
35 | | - | |
36 | | - | |
37 | | - | |
38 | | - | |
39 | | - | |
40 | | - | |
41 | | - | |
42 | | - | |
43 | | - | |
44 | | - | |
45 | | - | |
46 | 34 | | |
47 | | - | |
| 35 | + | |
48 | 36 | | |
49 | 37 | | |
50 | 38 | | |
| |||
117 | 105 | | |
118 | 106 | | |
119 | 107 | | |
120 | | - | |
121 | | - | |
122 | | - | |
| 108 | + | |
123 | 109 | | |
124 | 110 | | |
125 | 111 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
28 | 37 | | |
29 | 38 | | |
30 | 39 | | |
| |||
0 commit comments