Skip to content

Commit

Permalink
fix: tabIdex was missing for discard/save buttons (#453)
Browse files Browse the repository at this point in the history
Signed-off-by: Alejandro Parcet <[email protected]>
  • Loading branch information
AlexDygma authored Jun 26, 2023
1 parent 0c19554 commit 74cdfda
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 2 additions & 1 deletion src/renderer/component/Button/regularButton.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,14 +18,15 @@
import React from "react";
import PropTypes from "prop-types";

const RegularButton = ({ selected, onClick, size, buttonText, style, icoSVG, icoPosition, disabled }) => {
const RegularButton = ({ selected, onClick, size, buttonText, style, icoSVG, icoPosition, disabled, tabIndex }) => {
return (
<div
onClick={disabled ? () => {} : onClick}
className={`${size ? size : ""} ${selected ? "active" : ""} button ${style && style} iconOn${
icoPosition ? icoPosition : "None"
}`}
disabled={disabled}
tabIndex={tabIndex}
>
<div className={"buttonLabel"}>
{icoSVG && icoPosition !== "right" ? icoSVG : ""}
Expand Down
2 changes: 1 addition & 1 deletion src/renderer/modules/KeyPickerKeyboard/KeyPickerReduced.js
Original file line number Diff line number Diff line change
Expand Up @@ -480,7 +480,7 @@ class KeyPickerReduced extends Component {
icony={key.icony}
iconsize={key.iconsize}
disabled={key.mod == disableMods || key.move == disableMove || disableAll}
// disabled={false}
tabIndex={-1}
idArray={key.idArray}
keyCode={code}
/>
Expand Down
2 changes: 2 additions & 0 deletions src/renderer/modules/Saving/Saving.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,13 +27,15 @@ const Saving = ({ saveContext, destroyContext, inContext }) => {
return (
<Style className="savingButtons">
<RegularButton
tabIndex={0}
onClick={destroyContext}
buttonText={i18n.app.cancelPending.button}
style="outline"
size="sm"
disabled={!inContext}
/>
<RegularButton
tabIndex={0}
onClick={saveContext}
buttonText={i18n.components.save.button}
style="primary"
Expand Down

0 comments on commit 74cdfda

Please sign in to comment.