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

Alt+Up and Alt+Down do not display selected record #804

Open
fireundubh opened this issue Mar 30, 2020 · 1 comment
Open

Alt+Up and Alt+Down do not display selected record #804

fireundubh opened this issue Mar 30, 2020 · 1 comment
Assignees
Milestone

Comments

@fireundubh
Copy link
Contributor

fireundubh commented Mar 30, 2020

Summary

something broke with alt+up/down. if the cursor's focused on the record itself rather than record list on the left, alt+up/down doesn't change the displayed record

via Discord (yarrmateys#9065)

Conditions

  • xEdit 4.1.3b

Steps to Reproduce

  1. Activate the View frame.
  2. Press Alt+Up or Alt+Down to navigate the tree.

Result

Per TfrmMain.FormKeyDown starting at line 6290, we should be able to navigate the tree using arrow keys with the Alt modifier, but due to the fix for #798, this does not happen.

The selection changes but the record for the focused node is not displayed in the View frame.

We likely need to call TryViewOrCompareSelectedRecords in TfrmMain.FormKeyDown somewhere. Need to investigate further.

Addendum

We should implement functionality that allows Alt+Shift+Up and Alt+Shift+Down to select multiple records, too.

@fireundubh fireundubh changed the title Alt+Up and Alt+Down does not display selected record Alt+Up and Alt+Down do not display selected record Mar 30, 2020
@ElminsterAU ElminsterAU added this to the 4.1.x milestone Mar 31, 2020
@yarrmateys
Copy link

alt+arrow isn't the only thing that causes this problem. it may be related to the same bug though. on the list if you use something other than up/down, like a letter to find the plugins starting with it or home to go to the top, the display also won't refresh. say, you start on fallout4.esm and press d a few times to cycle through the dlc files.

ElminsterAU added a commit that referenced this issue Apr 3, 2020
#804: fixed issue where previously merged fix for #798 introduced node selection issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants