Skip to content

Conversation

@Pekos123
Copy link
Collaborator

No description provided.

@Pekos123 Pekos123 requested a review from Cosinus215 July 15, 2024 21:29
@DAXPL DAXPL self-requested a review July 16, 2024 16:17
@DAXPL DAXPL assigned DAXPL and Pekos123 and unassigned DAXPL Jul 16, 2024
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ta zmiana nie powinna się tu znaleźć

playerCamera = GetComponentInChildren<Camera>();
cController = GetComponent<CharacterController>();
stepSource = GetComponent<AudioSource>();
cameraSensitivity *= PlayerPrefs.GetFloat("MouseSensitivity");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cameraSensitivity = jakasBaseValue * PlayerPrefs.GetFloat("MouseSensitivity");

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Base value jest wyżej ustawiony jako 100f, przy deklarowaniu zmiennej

[SerializeField] private AudioMixer effectsVolume;
[SerializeField] private AudioMixer musicVolume;
[SerializeField] private VolumeProfile SampleSceneProfile;
[SerializeField] private VolumeProfile SampleSceneProfile1;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nazwy nic nie mówią

Copy link
Collaborator Author

@Pekos123 Pekos123 Jul 16, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Przepisałem nazwy, zapomniałem o tym wspomnieć, nazwy właśnie tych volume profile są default i sam nie wiem za co one odpowiadają
image
image

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wydziel do osobnej klasy

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To powinno być w jednej klasie z opcjami graficznymi

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Możesz dać do main menu.
Pomyśl o każdej funkcjonalności gdzie ona powinna być, gdzie to ma sens i gdzie logicznie będzie tego szukać

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants