WybeNetwork / VistaPanel-Themes

Some new themes for VistaPanel of MyOwnFreeHost
https://docs.wybenetwork.com/vistapanel-themes/#/
Creative Commons Attribution Share Alike 4.0 International
21 stars 15 forks source link

dark-discord and tickets #22

Open Stage4000 opened 1 year ago

Stage4000 commented 1 year ago

Hey there just wanted you to know that viewing an open ticket with the discord dark theme is broken. The colors make it so its near impossible to read any of the contents of the page, for some reason table cells are white (or near white) with white text.

Stage4000 commented 1 year ago

I fixed this locally with some CSS jank.

Stage4000 commented 1 year ago

So after looking into this further, this might not be limited to the discord dark theme. It appears that the table that contains the support ticket has it's CSS set inline. I set all tables and all children of tables to have an important background color and this appears to be doing the trick.

4yx commented 1 year ago

Nice! Would you be able to open a PR and contribute the fix upstream?

Stage4000 commented 1 year ago

My changes aren't really... PR-able... I just added some raw CSS to my advert area to override the white background in the tickets table. I have not had a chance to thoroughly test that this fix doesn't cause cancer elsewhere since it is kind of broad CSS, but here it is.

<style>
.table>thead>tr>th, .table>tbody>tr>th, .table>tfoot>tr>th, .table>thead>tr>td, .table>tbody>tr>td, .table>tfoot>tr>td {
    padding:8px;
        background-color: #2c2f33 !important;
}
td{background-color: #2c2f33 !important;}
.table{background-color: #2c2f33 !important;}
</style>
Stage4000 commented 1 year ago

I don't actually think the middle clause is needed, you may want to test further. I think that the classes 1&3 are relevant though.

Stage4000 commented 1 year ago

I think adding background-color: #2c2f33 !important; as an insertion on line 485 of style.css might do the trick.

4yx commented 1 year ago

I'll test myself and apply a patch shortly; thank you for reporting this and highlighting where it can be applied!

Stage4000 commented 1 year ago

No lie, just spent an hour setting up an exact copy of your CDN and then testing edits.