Enter Key and \| key seem to be swapped in the matrix. Melody96

[rockydbull]

When using the qmk config website I have to move the enter key to the |\ position and the |\ key to the enter key position for them to properly work on their correct keys IRL according to the ansi standard. This is with the melody96 hotswap ansi board with the 1u bottom row.

[noroadsleft]

@zunger-humu,

You added the LAYOUT_hotswap macro to the melody96. Does your keymap work as intended when compiled from a current QMK master, and do you use ANSI Enter or ISO Enter? I looked at the macros and the LAYOUT_hotswap macro doesn't assign KC_BSLS on your keymap to the same place in the matrix as the other two macros would do.

Would appreciate your insight on this.

[rockydbull]

@noroadsleft I think you may have replied to the wrong user (I also might be misreading this). I didnt make a layout. I am having the flipped |\ and enter key issue on the default layout with QMK. I am using Ansi enter for reference.

[zunger-humu]

Whoops -- sorry I missed your message earlier. I'm using it with ANSI Enter. I haven't compiled it in about three months, so I'm not sure if it works at current head, but there hasn't been anything changed that I know of which should affect this. The assignment of KC_BSLS to a different place was deliberate: when I applied the other two keymaps to a hotswap melody96, that key was mismapped. (As were a few others; that's why I added this keymap in the first place)

…

On Sat, Aug 3, 2019 at 6:46 AM rockydbull ***@***.***> wrote: @noroadsleft <https://github.com/noroadsleft> I think you may have replied to the wrong user (I also might be misreading this). I didnt make a layout. I am having the flipped |\ and enter key issue on the default layout with QMK. I am using Ansi enter for reference. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6471?email_source=notifications&email_token=AHNFBF6SHGISDZ3D73NL5XLQCWD3FA5CNFSM4II55UW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3POYAY#issuecomment-517925891>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHNFBF3M55TRBJV7Y2KZBSLQCWD3FANCNFSM4II55UWQ> .

[noroadsleft]

@zunger-humu Thanks for responding.

@rockydbull I requested zunger-humu's input because he's the user who submitted the Melody96 LAYOUT_hotswap macro to QMK, and I wanted to make sure that it was working for him. Sometimes a user will submit code and then discover it doesn't work correctly after their submission has been merged, so they fix it locally but then never submit those fixes back to us. zunger-humu has no history of doing this, but I wanted to make sure as I don't own the board in question.

Actually, looking over your initial post, @rockydbull, are you just flashing the default keymap that loads when you select melody96 as your keyboard in QMK Configurator?

[zunger-humu]

I'm typing on said keyboard right now, in fact. :)

@noroadleft, that would be my next question, too; the default melody96 keymap definitely didn't work for me on the hotswap. My keymap may be a bit on the intense side (the ability to type in Greek letters and math symbols and such) but you should definitely be able to use it, or build your own with the macros added.

(Which reminds me that I had some improvements to that keymap in mind, I should do those)

(ETA: I just did those and made a PR, and in the process verified that melody96:zunger does work fine at head)