combat-carousel icon indicating copy to clipboard operation
combat-carousel copied to clipboard

Players dont see the carousel by default

Open yjeroen opened this issue 2 years ago • 1 comments

Describe the bug Players dont see the carousel by default

To Reproduce Steps to reproduce the behavior:

  1. Enable carousel to start with combat image

  2. Both GM and Trusted player have the following sidebar setting: image

  3. Start combat by clicking on the Toggle Combat State icon: image

  4. GM sees correctly the carousel, with the scene buttons hidden: image

  5. Trusted Player does not see the carousel, scene buttons are still shown. image Important to note is that the sidebar icon DID change from gray to white.

Expected behavior I would have expected the Trusted Player to see the same as the GM.

Environment Info (please complete the following information): image

  • CC Version: 0.2.5
  • Browser: Chrome Version 97.0.4692.71 (Official Build) (64-bit)
  • OS: Windows 10

yjeroen avatar Jan 13 '22 10:01 yjeroen

Developer on Discord mentioned he has no intention to change this behavior.

If anyone else wants to change this behavior just for your world, add a custom.js file to your world and in your world.json add it to your scripts variable.

Add the following in your custom script:

Hooks.on("ready", async function () {

	// Set defeault client settings of Combat Carousel if the setting "Collapse Navigation Bar" is not activated (which is usually the default setting or if the module is not loaded)
	if ( ! game.settings.get('combat-carousel', 'collapseNav') ) {
		console.log('World script | '+game.world.id+' | Combat Carousel detected - setting GM preferred client settings.');
		game.settings.set('combat-carousel', 'collapseNav', true);
		game.settings.set('combat-carousel', 'openOnCombatCreate', true);
		game.settings.set('combat-carousel', 'carouselSize', 'sm');
		game.settings.set('combat-carousel', 'panOnClick', true);
		game.settings.set('combat-carousel', 'alwaysOnTop', true);
	}
	
});

yjeroen avatar Jan 13 '22 19:01 yjeroen