nextcloud / forms

📝 Simple form & survey app for Nextcloud
https://apps.nextcloud.com/apps/forms
GNU Affero General Public License v3.0
316 stars 96 forks source link

Icon color in navigation differs from text color #2240

Closed AndyScherzinger closed 2 weeks ago

AndyScherzinger commented 1 month ago

Please use the 👍 reaction to show that you are affected by the same issue. Please don't comment if you have no relevant information to add!

Describe the bug

2024-07-08 20_11_18-Daily Form - Forms - Nextcloud Daily – Mozilla Firefox

Icon is black, text is white

To Reproduce Steps to reproduce the behavior:

  1. master-server and master-forms
  2. OS is dark mode, Nc is light mode

Expected behavior same color for both

Screenshots see above

Nextcloud (please complete the following information):

Smartphone (please complete the following information):

Browser log

Open your console, reload your page and/or do the action leading to this issue and copy/paste the log in this thread.
How to access your browser console (Click to expand) # Chrome - Press either CTRL + SHIFT + J to open the “console” tab of the Developer Tools. - Alternative method: 1. Press either CTRL + SHIFT + I or F12 to open the Developer Tools. 3. Click the “console” tab. # Safari - Press CMD + ALT + I to open the Web Inspector. - See Chrome’s step 2. (Chrome and Safari have pretty much identical dev tools.) # IE9 1. Press F12 to open the developer tools. 2. Click the “console” tab. # Firefox - Press CTRL + SHIFT + K to open the Web console (COMMAND + SHIFT + K on Macs). - or, if Firebug is installed (recommended): 1. Press F12 to open Firebug. 2. Click on the “console” tab. # Opera 1. Press CTRL + SHIFT + I to open Dragonfly. 2. Click on the “console” tab.

Additional context Add any other context about the problem here.

Chartman123 commented 1 month ago

@AndyScherzinger Is it possible that this is a bug in NcListItem? We're using currentColor for filling the SVG in FormsAppNavigation

AndyScherzinger commented 1 month ago

I don't know or rater lack the deeper technical knowledge. Maybe you do @susnux?