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

Pihole network table row backgrounds fix #489

Merged
merged 1 commit into from
Apr 8, 2023

Conversation

sambartik
Copy link
Contributor

@sambartik sambartik commented Apr 8, 2023

theme-park.dev

  • I have read the contributing guideline and understand that I have made the correct modifications

  • PR's are done against the develop branch.


Bug fixes

  • When submitting bugfixes please show a before and after screenshot of the fix, and a description of what the fix does.

Description:

Removes the table row background color styling in network details tab since it provides informational value to the user. Other tables are kept untouched by this change.

Before:
before

After:
after

Benefits of this PR and context:

It fixes the intended behaviour of the app which was broken by the theme.
context

How Has This Been Tested?

Inline CSS browser

Source / References:

@GilbN GilbN merged commit 119841d into themepark-dev:develop Apr 8, 2023
GilbN added a commit that referenced this pull request May 19, 2023
* Bad interpreter error associated with the NPM release of 2.9.21 (#486)

* Fix Pi-Hole network details table colors (#489)

* Grafana 9.5.1 element support (#492)

Adding support for only a few elements in latest version of Grafana

* Added uptime kuma card css (#494)

---------

Co-authored-by: goodbyepavlyi 3 <[email protected]>
Co-authored-by: Samuel Bartík <[email protected]>
Co-authored-by: rg9400 <[email protected]>
Co-authored-by: Henry Whitaker <[email protected]>
GilbN added a commit that referenced this pull request May 29, 2023
* Bad interpreter error associated with the NPM release of 2.9.21 (#486)

* Fix Pi-Hole network details table colors (#489)

* Grafana 9.5.1 element support (#492)

Adding support for only a few elements in latest version of Grafana

* Added uptime kuma card css (#494)

* Use hashlib instead of git shas

* Fix Latest manifest if logic

---------

Co-authored-by: goodbyepavlyi 3 <[email protected]>
Co-authored-by: Samuel Bartík <[email protected]>
Co-authored-by: rg9400 <[email protected]>
Co-authored-by: Henry Whitaker <[email protected]>
GilbN added a commit that referenced this pull request Jan 25, 2024
* 1.13.8 (#496)

* Bad interpreter error associated with the NPM release of 2.9.21 (#486)

* Fix Pi-Hole network details table colors (#489)

* Grafana 9.5.1 element support (#492)

Adding support for only a few elements in latest version of Grafana

* Added uptime kuma card css (#494)

* Use hashlib instead of git shas

* Fix Latest manifest if logic

---------

Co-authored-by: goodbyepavlyi 3 <[email protected]>
Co-authored-by: Samuel Bartík <[email protected]>
Co-authored-by: rg9400 <[email protected]>
Co-authored-by: Henry Whitaker <[email protected]>

* Set line feed in fetch script

* 1.15.1 (#534)

* qbittorrent: 💄Fixes for #526

* Unraid: Fix login script github.io URL
Unraid: Fix the navbar scroll overflow.

* Move from subversion to git for pulling the qbittorrent webui (#530)

* Move from subversion to git for pulling the qbittorrent webui

* slight verbage changes

---------

Co-authored-by: Blake <[email protected]>

* vuetorrent curl github redirect

The curl of the latest vuetorrent repo returns a 301 redirect code and location https://api.github.com/repositories/181508472/releases/latest resulting in a `Cannot iterate over null (null)` when jq filters assets against the output from the original request.

---------

Co-authored-by: GilbN <[email protected]>
Co-authored-by: goodbyepavlyi 3 <[email protected]>
Co-authored-by: Samuel Bartík <[email protected]>
Co-authored-by: rg9400 <[email protected]>
Co-authored-by: Henry Whitaker <[email protected]>
Co-authored-by: Blake <[email protected]>
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