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

隐藏横幅banner后,子分区菜单被挡住了 #78

Closed
nothingshiru opened this issue May 31, 2024 · 4 comments · Fixed by #89
Closed

隐藏横幅banner后,子分区菜单被挡住了 #78

nothingshiru opened this issue May 31, 2024 · 4 comments · Fixed by #89
Labels
bug Something isn't working

Comments

@nothingshiru
Copy link

第一行分区按钮的子分区菜单显示在按钮上方,隐藏banner后就没有空间完整显示了。可不可以让子分区菜单像第二行分区按钮那样,显示在按钮下方?
屏幕截图 2024-05-31 094554

@festoney8 festoney8 added the bug Something isn't working label May 31, 2024
@festoney8
Copy link
Owner

这些二级菜单是动态创建的,位置由B站的代码计算出来,修改这个难度比较高。。

@festoney8 festoney8 added the help wanted Extra attention is needed label May 31, 2024
@nothingshiru
Copy link
Author

我现在用 .v-popover.is-top { top: 36px; } 能让菜单显示在下方。

分区按钮在不同页面宽度有3种不同高度(22px, 26px, 30px)。我尝试用下面这些规则把菜单调到合适的位置,但是不生效,控制台有报错但看不懂……

@media ( width < 1100px ) {
    .v-popover.is-top {
        top: 28px;
    }
}
@media ( 1100 <= width < 1701px ) {
    .v-popover.is-top {
        top: 32px;
    }
}
@media ( 1701 <= width ) {
    .v-popover.is-top {
        top: 36px;
    }
}

@festoney8
Copy link
Owner

festoney8 commented Jun 18, 2024

我现在用 .v-popover.is-top { top: 36px; } 能让菜单显示在下方。

分区按钮在不同页面宽度有3种不同高度(22px, 26px, 30px)。我尝试用下面这些规则把菜单调到合适的位置,但是不生效,控制台有报错但看不懂……

@media ( width < 1100px ) {
    .v-popover.is-top {
        top: 28px;
    }
}
@media ( 1100 <= width < 1701px ) {
    .v-popover.is-top {
        top: 32px;
    }
}
@media ( 1701 <= width ) {
    .v-popover.is-top {
        top: 36px;
    }
}

这一方法确实可以在单机实现菜单栏下置,但在脚本中兼顾多种分辨率设备,在它的计算方法未知时实现并不容易

下方样式,@media screen 的使用有一些问题,语法可参考 MDN

@festoney8
Copy link
Owner

festoney8 commented Jun 18, 2024

等一下,找到控制菜单下置的样式了ohhh,下一版本就更新进去,3 种不同高度好解决

@festoney8 festoney8 removed the help wanted Extra attention is needed label Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants