-
Notifications
You must be signed in to change notification settings - Fork 641
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
Added feature to auto detect the OS end-of-line (EOL) character.#7393 #7887
Conversation
632f11a
to
3b1bcc2
Compare
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.
- Delay the tooltip visibility. It is annoying when trying to switch.
- When copying the text, crlf is not respected.
- Copy a crlf text from outside and paste in query tool. It should change based on current EOL. Vice-versa.
web/pgadmin/tools/sqleditor/static/js/components/sections/StatusBar.jsx
Outdated
Show resolved
Hide resolved
web/pgadmin/tools/sqleditor/static/js/components/sections/StatusBar.jsx
Outdated
Show resolved
Hide resolved
web/pgadmin/tools/sqleditor/static/js/components/sections/StatusBar.jsx
Outdated
Show resolved
Hide resolved
web/pgadmin/tools/sqleditor/static/js/components/sections/StatusBar.jsx
Outdated
Show resolved
Hide resolved
9fa0e3b
to
dbab371
Compare
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 will commit a fix for status CSS issue. Update the screenshot again.
web/pgadmin/static/js/components/ReactCodeMirror/CustomEditorView.js
Outdated
Show resolved
Hide resolved
web/pgadmin/static/js/components/ReactCodeMirror/components/Editor.jsx
Outdated
Show resolved
Hide resolved
web/pgadmin/tools/sqleditor/static/js/components/sections/Query.jsx
Outdated
Show resolved
Hide resolved
web/pgadmin/tools/sqleditor/static/js/components/sections/StatusBar.jsx
Outdated
Show resolved
Hide resolved
web/pgadmin/tools/sqleditor/static/js/components/sections/StatusBar.jsx
Outdated
Show resolved
Hide resolved
dbab371
to
08837aa
Compare
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 didn't find any code related to detecting a files EOL. OS level EOL is not enough. A file with only CRLF can be opened on a Mac env.
web/pgadmin/tools/sqleditor/static/js/components/sections/StatusBar.jsx
Outdated
Show resolved
Hide resolved
web/pgadmin/tools/sqleditor/static/js/components/sections/StatusBar.jsx
Outdated
Show resolved
Hide resolved
08837aa
to
f40c160
Compare
f40c160
to
e4b1e83
Compare
e4b1e83
to
246dd8a
Compare
Detail :- #7393