Various PF fixes

pkg/lib/cockpit-components-modifications.jsx

@@ -88,10 +88,10 @@ class ModificationsExportDialog extends React.Component {
                    title={_("Automation script") }>
                 <Tabs activeKey={this.state.active_tab} onSelect={this.handleSelect}>
                     <Tab eventKey="shell" title={_("Shell script")}>
-                        <TextArea resizeOrientation='vertical' isReadOnly defaultValue={this.props.shell.trim()} />
+                        <TextArea resizeOrientation='vertical' readOnlyVariant="default" defaultValue={this.props.shell.trim()} />
                     </Tab>
                     <Tab eventKey="ansible" title={_("Ansible")}>
-                        <TextArea resizeOrientation='vertical' isReadOnly defaultValue={this.props.ansible.trim()} />
+                        <TextArea resizeOrientation='vertical' readOnlyVariant="default" defaultValue={this.props.ansible.trim()} />
                         <div className="ansible-docs-link">
                             <OutlinedQuestionCircleIcon />
                             { _("Create new task file with this content.") }

pkg/lib/cockpit-components-table.jsx

@@ -188,7 +188,7 @@ export const ListingTable = ({
     const rowsComponents = (isSortable ? (sortMethod ? sortMethod(rows, activeSortDirection, activeSortIndex) : sortRows()) : rows).map((row, rowIndex) => {
         const rowProps = row.props || {};
         if (onRowClick) {
-            rowProps.isHoverable = true;
+            rowProps.isClickable = true;
             rowProps.onRowClick = (event) => onRowClick(event, row);
         }
 

pkg/selinux/setroubleshoot-view.jsx

@@ -120,7 +120,7 @@ class SELinuxEventDetails extends React.Component {
 
             // One line usually means one command
             if (itm.doText && itm.doText.indexOf("\n") < 0)
-                doElement = <TextArea aria-label={_("solution")} isReadOnly defaultValue={itm.doText} />;
+                doElement = <TextArea aria-label={_("solution")} readOnlyVariant="default" defaultValue={itm.doText} />;
 
             // There can be text with commands. Command always starts on a new line with '#'
             // Group subsequent commands into one `<TextArea>` element.
@@ -142,7 +142,7 @@ class SELinuxEventDetails extends React.Component {
                 });
                 doElement = parts.map((p, index) => p[0] == "#"
                     ? <TextArea aria-label={_("solution")}
-                                isReadOnly
+                                readOnlyVariant="plain"
                                 key={p}
                                 defaultValue={p.substr(2)} />
                     : <span key={p}>{p}</span>);

pkg/shell/superuser.jsx

@@ -203,7 +203,7 @@ const UnlockDialog = ({ proxy, host }) => {
             <Form isHorizontal>
                 <FormGroup fieldId="switch-to-admin-access-bridge-select"
                            label={_("Method")}>
-                    <FormSelect value={method} onChange={(_, method) => setMethod(method)} isDisabled={busy}>
+                    <FormSelect id="switch-to-admin-access-bridge-select" value={method} onChange={(_, method) => setMethod(method)} isDisabled={busy}>
                         { methods.map(m => <FormSelectOption value={m} key={m}
                                                              label={_(proxy.Methods[m].v.label.v)} />) }
                     </FormSelect>

pkg/systemd/abrtLog.jsx

@@ -34,7 +34,7 @@ import { journal } from "journal";
 
 const _ = cockpit.gettext;
 
-const Table = ({ lines, delimiter, type }) => {
+const Table = ({ lines, delimiter }) => {
     return (
         <DescriptionList className="pf-m-horizontal-on-sm">
             { lines.map((line, idx) => {