Skip to content
This repository has been archived by the owner on Jul 12, 2024. It is now read-only.

WIP: added troubleshooting page #6

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

zenoVerboom
Copy link
Contributor

please let me know your opinion on the new functionality.

- trouble shooter added
- icon changed for satanicflame
- corrected some typo's
- added shortcut to troubleshooter on home page
-added instruction install video
-added new item in troubleshooter
- mic sens
- other quick fixes
forgot to run the f*cking thingy
- added gif how to configure  bedrock token
- added local jquery instead of linking
Copy link
Member

@Mindgamesnl Mindgamesnl left a comment

Choose a reason for hiding this comment

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

Hey there!
I went over everything, but some things stick out as odd

@@ -15,4 +15,5 @@ These browsers are fully supported in the latest OpenAudioMc release.

# Discontinued Browsers
These browsers are not supported (anymore) by OpenAudioMc. If you got redirected to this page, then that probably means that you were using one of these. Please try to open your client with a different browser. (tip: you can click "Copy to clipboard" in game to manually open it in a supported browser)
- Internet Explorer: *Got replaced by Edge*
- [Internet Explorer](https://www.microsoft.com/nl-be/download/internet-explorer.aspx): *Got replaced by Edge*
- All mobile browsers like [Samsung Internet](https://play.google.com/store/apps/details?id=com.sec.android.app.sbrowser&hl=nl&gl=US)
Copy link
Member

Choose a reason for hiding this comment

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

It isn't technically discontinued, we never supported it, to begin with.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

true, I just added it since it was technically working before without voicechat. Made it clear by adding title unsupported browsers

content/geyser.md Outdated Show resolved Hide resolved
content/hue.md Outdated Show resolved Hide resolved
@@ -3,7 +3,7 @@
[//]: # (DESCRIPTION:Extra perks of becoming a sponsor)
[//]: # (TAGS:regions,areas,worldguard,world,region,media,accounts)

OpenAudioMc has always been a hobby project that I ran in my spare time ever since it launched <b>5 years</b> ago.
OpenAudioMc has always been a hobby project that I ran in my spare time ever since it launched `5 years` ago.
Copy link
Member

Choose a reason for hiding this comment

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

whats wrong with bold?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lets say, all pages make use of the blue shiny highlight, this is the only page you used bold. This was a suggestion from IOByte, but feel free to change it back if you would like to

@@ -0,0 +1,2 @@
/*! jQuery v3.6.0 | (c) OpenJS Foundation and other contributors | jquery.org/license */
Copy link
Member

Choose a reason for hiding this comment

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

Why add jQuery as a dependency? it doesn't add much nowadays

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well you could blame my school for that, we were forced to use it. I used just plain javascript and got a 0. (And no it wasn't like years ago ...) Now after using it a lot I find it clearer to read for the troubleshooter.

cursor: pointer;
box-shadow: 0 3px 6px rgba(145, 145, 145, 0.16), 0 3px 6px rgba(89, 89, 89, 0.23);
}
.bg-troubleshooter{
Copy link
Member

Choose a reason for hiding this comment

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

added a comment here about the troubleshooting.png asset, why not just use the logo with padding instead of forcing an asset like that?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is actually done like that. The asset was just a placeholder for developing it. Forgot to remove it.

@Mindgamesnl Mindgamesnl changed the title added troubleshooting page WIP: added troubleshooting page Jan 13, 2022
Copy link
Contributor Author

@zenoVerboom zenoVerboom left a comment

Choose a reason for hiding this comment

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

Hi, sorry there were some issues with it. hope it is useful now after some tweaks. Would be nice if we had a little better communication next time, that would have solved a lot already

@@ -15,4 +15,5 @@ These browsers are fully supported in the latest OpenAudioMc release.

# Discontinued Browsers
These browsers are not supported (anymore) by OpenAudioMc. If you got redirected to this page, then that probably means that you were using one of these. Please try to open your client with a different browser. (tip: you can click "Copy to clipboard" in game to manually open it in a supported browser)
- Internet Explorer: *Got replaced by Edge*
- [Internet Explorer](https://www.microsoft.com/nl-be/download/internet-explorer.aspx): *Got replaced by Edge*
- All mobile browsers like [Samsung Internet](https://play.google.com/store/apps/details?id=com.sec.android.app.sbrowser&hl=nl&gl=US)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

true, I just added it since it was technically working before without voicechat. Made it clear by adding title unsupported browsers

content/geyser.md Outdated Show resolved Hide resolved
content/hue.md Outdated Show resolved Hide resolved
@@ -3,7 +3,7 @@
[//]: # (DESCRIPTION:Extra perks of becoming a sponsor)
[//]: # (TAGS:regions,areas,worldguard,world,region,media,accounts)

OpenAudioMc has always been a hobby project that I ran in my spare time ever since it launched <b>5 years</b> ago.
OpenAudioMc has always been a hobby project that I ran in my spare time ever since it launched `5 years` ago.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Lets say, all pages make use of the blue shiny highlight, this is the only page you used bold. This was a suggestion from IOByte, but feel free to change it back if you would like to

cursor: pointer;
box-shadow: 0 3px 6px rgba(145, 145, 145, 0.16), 0 3px 6px rgba(89, 89, 89, 0.23);
}
.bg-troubleshooter{
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is actually done like that. The asset was just a placeholder for developing it. Forgot to remove it.

@@ -0,0 +1,2 @@
/*! jQuery v3.6.0 | (c) OpenJS Foundation and other contributors | jquery.org/license */
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Well you could blame my school for that, we were forced to use it. I used just plain javascript and got a 0. (And no it wasn't like years ago ...) Now after using it a lot I find it clearer to read for the troubleshooter.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants