Column header with blue outline after clicking for sorting

[oim5nu]

What version of React-Table are you using?

Your bug may already be fixed in the latest release. Run yarn upgrade react-table!
6.7.5

What bug are you experiencing, or what feature are you proposing?

As described in Title.
Refer to description by @gary-menzel in #682

Use https://codesandbox.io/s/X6npLXPRW (by clicking the "Fork" button) to reproduce the issue.

https://codesandbox.io/s/5knzrn8qmk

What are the steps to reproduce the issue?

1. npm install -g create-react-app
2. Add 'react-table@6.7.5' to package.json.
3. replace src/App.js with following code

import React, { Component } from 'react';
import logo from './logo.svg';
import './App.css';
import ReactTable from 'react-table';
import 'react-table/react-table.css';

class App extends Component {
  constructor() {
    super();
  }
  render() {
    const data = [
            {
                "location":"01-01",
                "amount":150
            },
            {
                "location":"01-02",
                "amount":3000
            },
            {
                "location":"01-03",
                "amount":1201
            }
    ];

    const columns = [
      {
        Header: "Location",
        accessor: "location"
      },
      {
        Header: "Amount",
        accessor: "amount",
        Cell: props => <span className='number'>{props.value}</span>
      }
    ];

    return (
      <div className="App">
        <header className="App-header">
          <img src={logo} className="App-logo" alt="logo" />
          <h1 className="App-title">Welcome to React</h1>
        </header>
        <p className="App-intro">
          To get started, edit <code>src/App.js</code> and save to reload.
        </p>
        <ReactTable
          data={data}
          columns={columns}
          classsName='-striped -highlight'
        />
      </div>
    );
  }
}

export default App;

4. npm start.
5. Click any title of headers, and it shows

[gary-menzel]

Thanks for that - checking in some changes against this issue now. Not sure how quickly it will make it into a release (but I have notified the owner).

[caub]

Yes noticed it too, the outline also wraps the column-resizer so it looks weird

I think the outline is fine, just fixing the boundaries will be good (by moving resize handles out of focusable header buttons)

[gary-menzel]

The focus issue is a bug that has a PR in waiting to get released.

[maccuaa]

I've recently started using React Table and noticed that this bug is still occurring. Using the Code Sandbox link referenced above (https://codesandbox.io/s/5knzrn8qmk) I updated the react-table dependency to 6.8.0 and the blue outline is still there.

I've create a PR with a fix (see #881)

[gary-menzel]

Please see my comment on the PR.

[alexpreynolds]

One thing that works for me is using the onSortedChange property, overriding it to blur the document's active element.

<ReactTable
    ...
    onSortedChange={(c, s) => { document.activeElement.blur() }}
    ...
    />

The blue outline flickers so it isn't eliminated completely, but at least it doesn't hang around and be a distracting nuisance for people who do not need to track focus.

Hope this helps others!

[Jefski14]

Any news on that PR? I still have the Issue in 6.8.2

[mikkel-vagn]

Add this to your CSS:

.ReactTable .rt-thead [role="columnheader"] {
outline: 0;
}

[cedricholz]

I updated react-table which didn't work and I couldn't figure out how to add Mikkel-Vagn's css in without hardcoding it into the module files. This works for me though.

getTheadThProps={() => { return { style: { outline: 0, } }; }}

[claidler]

Following on from @cedricholz comment, incase this is of any use to anyone - I had grouped table headers, so used the getTheadGroupThProps function also:

getTheadThProps={() => { return { style: { outline: 0, } }; }}
getTheadGroupThProps = {() => { return { style: { outline: 0, } }; }}

[dukekosy]

.ReactTable .rt-resizer{ display:inline-block; position:absolute; width:3px; top:0; bottom:0; right:-1px; cursor:col-resize; z-index:10 }
did the trick. it reduces the size of the table column resizer.