Uploaded image for project: 'JCommune'
  1. JCommune
  2. JC-1206

Place new comment to code review below last comment on current line

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Duplicate
    • Affects Version/s: 1.1 Larks
    • Fix Version/s: backlog
    • Labels:
      None

      Description

      It makes sense to place new comment to code review below last comment on current line.

      Now we typing comment above last one (on current line) but when we send it - our comment displays below last one.

        Attachments

          Issue Links

            Activity

            Hide
            varro Artem R added a comment -

            Improvement for handy layout of comments for Code Review

            Show
            varro Artem R added a comment - Improvement for handy layout of comments for Code Review
            Hide
            ctapobep Stanislav Bashkyrtsev added a comment -

            What if we have a lot of comments? Then after pressing on the row, the input will appear at the bottom where user doesn't see it. Neither she'll be able to see the line of code she's commenting.

            Show
            ctapobep Stanislav Bashkyrtsev added a comment - What if we have a lot of comments? Then after pressing on the row, the input will appear at the bottom where user doesn't see it. Neither she'll be able to see the line of code she's commenting.
            Hide
            varro Artem R added a comment -

            Yes. But when we have a lot of comments maybe we more frequent need to see last comments (it is newer info, we don't read it before) than code string?

            Show
            varro Artem R added a comment - Yes. But when we have a lot of comments maybe we more frequent need to see last comments (it is newer info, we don't read it before) than code string?
            Hide
            ctapobep Stanislav Bashkyrtsev added a comment -

            Will be fixed in scope of JC-1333

            Show
            ctapobep Stanislav Bashkyrtsev added a comment - Will be fixed in scope of JC-1333

              People

              • Assignee:
                Unassigned
                Reporter:
                varro Artem R
              • Votes:
                1 Vote for this issue
                Watchers:
                Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Structure Helper Panel