onIsFullScreenChange prevents toggle icon from changing

[nimo23]

I use enableFullScreenToggle: true

and have

onIsFullScreenChange: (event) => {
	console.info(event);
        // BUG? will always be false when using onIsFullScreenChange-event
        console.log(table.getState().isFullScreen); 
      ..
},
muiTablePaperProps: ({ table }) => ({
	style: {
		zIndex: table.getState().isFullScreen ? 1000 : undefined,
	   }
	}),

When toggling full-screen mode, onIsFullScreenChange is called successfully, but

1. the full-screen toggle icon is not changed
2. the custom zIndex: table.getState().isFullScreen ? 1000 : undefined, will not be applied when using onIsFullScreenChange.
3. and the styles defined at https://www.material-react-table.com/docs/guides/full-screen-toggle#change-z-index-of-full-screen-table are not added

To point 1:
When toggling the full-screen icon, the icon should toggle even if the user defines a custom onIsFullScreenChange event - this should always happen because it makes sense.

To point 2:
table.getState().isFullScreen should also return true even if an onIsFullScreenChange event is used.

To point 3:
Although I understand this behavior of not adding styles because we might be applying something else by using onIsFullScreenChange, it would be good to have a method that calls the default fullscreen method (and does not prevent the default method), e.g.

onIsFullScreenChange: (event, state: Updater<boolean>) => {
	console.info(state); 
       // to not prevent react-material-table from applying its style and setting isFullScreen
       event.preventDefault(false); 
},

or

onIsFullScreenChange: (event, table) => {
	console.info(state); 
       //  So that react-material-table can apply its style and setting isFullScreen.
       // However, this will cause recursion issue as onIsFullScreenChange will be called again and again..:
       table.setIsFullscreen(true);
},

[KevinVandy]

Follow the typescript types and you won't get lost. Also, read the state management docs

[nimo23]

@KevinVandy Thank you! I managed it accordingly https://www.material-react-table.com/docs/guides/state-management#manage-individual-states-as-needed.