Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New dark mode look and Minecraftia font boxes #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

A44690
Copy link

@A44690 A44690 commented Mar 13, 2024

The page will be in dark mode if the computer is in dark mode. The font of texts under "Connection" box, "Whitelist" box, and "Blacklist" box will all now be Minecraftia.

The page will be in dark mode if the computer is in dark mode.
The font of texts under "Connection" box, "Whitelist" box, and "Blacklist" box will all now be Minecraftia.
@@ -244,7 +247,7 @@ onMounted(async () => {
</div>
</fieldset>
<div v-else>
<i><b>Loading...</b></i>
<!--<i><b>Loading...</b></i>-->Loading...
Copy link
Member

Choose a reason for hiding this comment

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

你怎么把我的 Loading 删掉了

Copy link
Author

Choose a reason for hiding this comment

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

字体不对?
怎么改

Copy link
Member

Choose a reason for hiding this comment

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

字体不对不是删掉的理由 😅

给它加个 class, 然后 css 里面设置 font-family 就好

Copy link
Author

Choose a reason for hiding this comment

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

我直接改的是

Loading...

Copy link
Author

Choose a reason for hiding this comment

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

Loading...

Copy link
Author

Choose a reason for hiding this comment

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

byd这行代码github打不出来?

Copy link
Author

Choose a reason for hiding this comment

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

Loading...

Copy link
Author

Choose a reason for hiding this comment

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

我。。。

@@ -111,6 +111,7 @@ async function refresh(): Promise<void> {

var loginPromise: Promise<void> | null = null

/*whoever don't have an account butwant to check how it looks delete this function*/
Copy link
Member

Choose a reason for hiding this comment

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

我不建议在这里加这个 comment, 之后我加个 dev api 吧

@@ -224,7 +225,9 @@ onMounted(async () => {
<legend class="config-box-title">
<h3>Connections</h3>
</legend>
<div v-if="connPollError" class="error">{{ connPollError }}</div>
<!-- replacement code to check whether error visual works
<div v-if= "false||connPollError" class="error">{{ connPollError }}</div> -->
Copy link
Member

Choose a reason for hiding this comment

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

缩进要对齐

}
}

/*main resources*/
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/*main resources*/
/* main resources */

box-shadow: 0 0 2px #000a;
}

@media(max-width: 45rem) {
Copy link
Member

Choose a reason for hiding this comment

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

? 你怎么给我适配 mobile 的 media 删了

Copy link
Author

Choose a reason for hiding this comment

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

移到上面了

Copy link
Member

Choose a reason for hiding this comment

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

@media放最下面,因为按照 css 加载顺序来说,下面的值会覆盖上面的值

就是让代码更好看

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.

2 participants