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

CSS displaying font color differently on mobile (white text on white background) #10191

Closed
2 tasks done
aranar-pro opened this issue Oct 12, 2021 · 1 comment
Closed
2 tasks done

Comments

@aranar-pro
Copy link
Contributor

Debug mode

Describe the bug

Side by side comparison of Desktop Browser to Mobile (safari) showing different font color for "dropdown-menu" items (Create New). Tested in versions 5.1.8 (Windows/IIS) and 6.0 (Demo site).
Desktop: .skin-blue .main-header .navbar .dropdown-menu li {color: #333} from all.css (dark text on white background)
Mobile: .skin-blue .main-header .navbar .nav > li > a : {color: #fff} from skin-blue.css (White Text on White background)

Reproduction steps

  1. Open https://develop.snipeitapp.com on Mobile (Safari iOS, Crome for iOS)
  2. Click "Create New" drop down menu.
  3. Text appears white on a white background.

Expected behavior

Text in dropdown-menu should be black on a white background similar to desktop site.

Screenshots

Mobile: IMG_1897
Desktop:
Screen Shot 2021-10-12 at 12 19 01 PM

Snipe-IT Version

5.1.8 & 6

Operating System

Windows

Web Server

IIS

PHP Version

7.4

Operating System

MacOS

Browser

Safari

Version

14

Device

iPhoneXS

Operating System

14.8

Browser

Chrome (iOS) , Safari (iOS)

Version

94.04606.76, 14.8

Error messages

In Desktop Browser, Demo site (v6) produces following errors:

[Error] Failed to load resource: the server responded with a status of 404 (Not Found) (bootstrap.css.map, line 0)
[Error] Failed to load resource: the server responded with a status of 404 (Not Found) (ekko-lightbox.js.map, line 0)

Additional context

Demo site v6 (confirmed on personal site 5.1.8 IIS, PHP 7.4)

@Godmartinz
Copy link
Collaborator

Godmartinz commented Feb 23, 2022

#10728

snipe added a commit that referenced this issue Mar 19, 2022
commit d55c176
Merge: 93ff952 c0451fe
Author: snipe <[email protected]>
Date:   Wed Mar 16 18:35:42 2022 +0000

    Merge pull request #10831 from snipe/merges/master_down_to_develop_march_16

    v6.0.0-RC-5 - Merges master down to develop

commit c0451fe
Author: snipe <[email protected]>
Date:   Wed Mar 16 18:10:17 2022 +0000

    Bumped version to v6.0.0-RC-5

    Signed-off-by: snipe <[email protected]>

commit 0f95802
Author: snipe <[email protected]>
Date:   Wed Mar 16 18:05:47 2022 +0000

    Fixed mis-automerge

    Signed-off-by: snipe <[email protected]>

commit 9db8bd7
Author: snipe <[email protected]>
Date:   Wed Mar 16 18:02:07 2022 +0000

    Merging master down into develop

    Signed-off-by: snipe <[email protected]>

commit 93ff952
Merge: 40a9470 89ddbdd
Author: snipe <[email protected]>
Date:   Wed Mar 16 10:04:58 2022 -0700

    Merge pull request #10829 from snipe/features/add_statuslabel_filter_by_type_in_api

    Added filter by status_type in StatusLabels API index endpoint

commit 89ddbdd
Author: snipe <[email protected]>
Date:   Wed Mar 16 16:53:18 2022 +0000

    Fixed comment

    Signed-off-by: snipe <[email protected]>

commit 7498fe3
Author: snipe <[email protected]>
Date:   Wed Mar 16 16:45:03 2022 +0000

    Removed extra space because pedantry

    Signed-off-by: snipe <[email protected]>

commit babf7c0
Author: snipe <[email protected]>
Date:   Wed Mar 16 16:38:45 2022 +0000

    Added ability to filter status label index endpoint by status type

    Signed-off-by: snipe <[email protected]>

commit 40a9470
Merge: f499dcf 781b426
Author: snipe <[email protected]>
Date:   Wed Mar 16 08:54:10 2022 -0700

    Merge pull request #10828 from snipe/fixes/10826_parse_error_on_print_all

    Fixed #10826 - parse error on translation string for print all assets

commit 781b426
Author: snipe <[email protected]>
Date:   Wed Mar 16 08:50:47 2022 -0700

    Fixed #10826 - parse error on translation string for print all assets

    Signed-off-by: snipe <[email protected]>

commit f499dcf
Merge: 84aa26d 54d6a4d
Author: snipe <[email protected]>
Date:   Wed Mar 16 08:25:29 2022 -0700

    Merge pull request #10827 from snipe/fixes/modal_dropdowns_broken

    Fixed #10825 - selectlists in modals b0rked

commit 54d6a4d
Author: snipe <[email protected]>
Date:   Wed Mar 16 08:23:50 2022 -0700

    Fixed #10825 - selectlists in modals b0rked

    Signed-off-by: snipe <[email protected]>

commit 84aa26d
Merge: 8984b3a 14bd07e
Author: snipe <[email protected]>
Date:   Thu Mar 10 13:02:45 2022 -0800

    Merge pull request #10728 from Godmartinz/gh10191-css-dropdownmenu

    Fixed #10191 - font color contrast on mobile menu

commit 8984b3a
Merge: d06ef4b cdc0805
Author: snipe <[email protected]>
Date:   Thu Mar 10 10:40:33 2022 -0800

    Merge pull request #10812 from inietov/fixes/CarbonExceptions_InvalidFormatException_DateTime_develop

    Fixes Carbon\Exceptions\InvalidFormatException: DateTime::__construct(): Failed to parse time string develop

commit cdc0805
Author: Ivan Nieto Vivanco <[email protected]>
Date:   Thu Mar 10 12:07:07 2022 -0600

    Sanitize dates input in the importer before saving

commit 14bd07e
Merge: 4435ea9 16fd109
Author: Godfrey M <[email protected]>
Date:   Thu Mar 3 10:14:30 2022 -0800

    gerge branchevelop' into gh10191-css-dropdownmenu

commit 4435ea9
Author: Godfrey M <[email protected]>
Date:   Thu Mar 3 09:47:11 2022 -0800

    removes changes to mix-manifest

commit f115385
Author: Godfrey M <[email protected]>
Date:   Thu Mar 3 09:32:05 2022 -0800

    C&P mix-manifest to be tracked

commit 708dc08
Author: Godfrey M <[email protected]>
Date:   Wed Feb 23 11:17:44 2022 -0800

    mixes color contrast on dropdown menu from navbar
    on file

Signed-off-by: snipe <[email protected]>
@snipe snipe closed this as completed in 84aa26d May 13, 2022
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

No branches or pull requests

2 participants