-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
open file at a specific line using copied link/URL #308
Conversation
Signed-off-by: Adarshdeep Cheema <[email protected]>
Signed-off-by: Adarshdeep Cheema <[email protected]>
Signed-off-by: Adarshdeep Cheema <[email protected]>
Signed-off-by: Adarshdeep Cheema <[email protected]>
…t top and copy the DataSet line content Signed-off-by: Adarshdeep Cheema <[email protected]>
Signed-off-by: Adarshdeep Cheema <[email protected]>
webClient/src/app/editor/code-editor/monaco/monaco.component.ts
Outdated
Show resolved
Hide resolved
webClient/src/app/editor/code-editor/monaco/monaco.component.ts
Outdated
Show resolved
Hide resolved
webClient/src/app/editor/code-editor/monaco/monaco.component.ts
Outdated
Show resolved
Hide resolved
webClient/src/app/editor/code-editor/monaco/monaco.component.ts
Outdated
Show resolved
Hide resolved
Signed-off-by: Adarshdeep Cheema <[email protected]>
Signed-off-by: Adarshdeep Cheema <[email protected]>
Signed-off-by: AdarshdeepCheema <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not able to remove the highlight from the page once it's on there. I can remove it, if the line is empty but if I delete the line, and type over it, the highlight comes back. The highlight can also be shifted around using linebreaks (see screenshot). I think you're opening a can of worms here to bug test this UX, I would recommend just sticking to highlighting the line number instead of the line itself. The rest of the PR I love, the UX just feels buggy
I think you resolved this but forgot to remove the 2nd snackbar: |
Signed-off-by: Adarshdeep Cheema <[email protected]>
Changed the decoration to the line number instead of line content and now we will remove the decoration if the user tries to edit the file Screen.Recording.2022-11-17.at.11.17.39.AM.mov |
Fixed/Removed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes addressed, this looks great. We just need to document this now @AdarshdeepCheema
Signed-off-by: Adarshdeep Cheema [email protected]
Proposed changes
This PR addresses Issue: [Link to Github issue within https://github.com/zowe/zlux/issues if any]
This PR depends upon the following PRs:
Type of change
Please delete options that are not relevant.
PR Checklist
Please delete options that are not relevant.
Testing
Further comments