Skip to content
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

PR: Add scroll line up/down keyboard shortcuts (Editor) #10990

Merged
merged 13 commits into from
Aug 4, 2023

Conversation

jnsebgosselin
Copy link
Member

@jnsebgosselin jnsebgosselin commented Dec 9, 2019

Description of Changes

  • Wrote at least one-line docstrings (for any new functions)
  • Added unit test(s) covering the changes (if testable)
  • Included a screenshot or animation (if affecting the UI, see Licecap)

scroll_with_keyboard

image

Issue(s) Resolved

Fixes #10968

Affirmation

By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: Jean-Sébastien Gosselin

@jnsebgosselin jnsebgosselin added this to the v5.0beta1 milestone Dec 9, 2019
@jnsebgosselin jnsebgosselin self-assigned this Dec 9, 2019
spyder/config/main.py Outdated Show resolved Hide resolved
@goanpeca goanpeca removed this from the v5.0beta1 milestone Feb 18, 2020
…plement_scoll_line_sc

# Conflicts:
#	spyder/plugins/editor/widgets/tests/test_editor.py
@jnsebgosselin
Copy link
Member Author

So as I've said in gitter, I'm currently unable to work on master after PR #11064 because of the following error refusing to allow an OAuth App to create or update workflow `.github/workflows/duplicates.yml` without `workflow` scope. So that is why this branch is not up-to-date yet. I'll update it as soon as I find a working solution to this problem.

@ccordoba12 ccordoba12 added this to the v5.0alpha2 milestone Aug 5, 2020
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha2, v5.0alpha3 Nov 12, 2020
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha3, v5.0alpha4 Jan 8, 2021
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha4, v5.0alpha5 Feb 14, 2021
@ccordoba12 ccordoba12 modified the milestones: v5.0alpha5, v5.0alpha6 Feb 23, 2021
@ccordoba12 ccordoba12 removed this from the v5.0alpha6 milestone Mar 19, 2021
@ccordoba12 ccordoba12 added this to the v5.0beta1 milestone Mar 19, 2021
@ccordoba12 ccordoba12 modified the milestones: v5.0.0, v5.x Apr 2, 2021
@dalthviz dalthviz modified the milestones: v5.x, v6.0alpha2 Jul 31, 2023
@dalthviz
Copy link
Member

dalthviz commented Jul 31, 2023

Hi @jnsebgosselin sorry for the sudden ping, but just wondering, is okay if I go ahead and finish this? Let me know!

Edit: Will push a commit trying to fix the conflicts here

@dalthviz dalthviz self-assigned this Aug 1, 2023
@ccordoba12 ccordoba12 changed the title PR: Add scroll line up/down using keyboard shortcut PR: Add scroll line up/down using keyboard shortcuts Aug 3, 2023
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment for you @dalthviz, the rest looks good to me.

@ccordoba12 ccordoba12 changed the title PR: Add scroll line up/down using keyboard shortcuts PR: Add scroll line up/down keyboard shortcuts Aug 3, 2023
@jnsebgosselin
Copy link
Member Author

@dalthviz awesome, thanks for finishing this!

@dalthviz dalthviz changed the title PR: Add scroll line up/down keyboard shortcuts PR: Add scroll line up/down keyboard shortcuts (Editor) Aug 3, 2023
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jnsebgosselin for your here and @dalthviz for finishing it.

@ccordoba12 ccordoba12 merged commit 900fee5 into spyder-ide:master Aug 4, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: scroll line up/down using keyboard shortcut
4 participants