The text color in the search form text field is inheriting Bartik's header text color (#fff) and id therefore invisible against the white text field background.

Worked properly when I added this in chrome inspector:

div.form-item.form-type-search.form-item-keys {
  color: #333;
}

Comments

tkoleary’s picture

Issue summary: View changes
Saphyel’s picture

Component: search.module » Bartik theme
Assigned: Unassigned » Saphyel
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new464 bytes
new351 bytes

This is my first contributed patch for drupal core and I did in "Sprint day Zaragoza"

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Hi @Saphyel thanks for your first patch! Nice work. After a patch is submitted the status should be changed to "needs review". The review process is documented here https://www.drupal.org/contributor-tasks/review. An easy rule to remember is that you don't rtbc your own patches. You could help the reviewer by providing before and after screenshot.

+++ b/core/themes/bartik/css/colors.css
@@ -60,3 +60,6 @@ a:active,
\ No newline at end of file

Need a new line at the end of the file

emma.maria’s picture

Version: 8.0-alpha13 » 8.0.x-dev
Assigned: Saphyel » Unassigned
Issue tags: -D8UX usability +frontend, +CSS, +Novice, +Needs screenshots, +Needs reroll
adci_contributor’s picture

StatusFileSize
new4.62 KB
new375 bytes

Is it still actual to the current Bartik? When I tried to reproduce, text-color in a search form on the header region was #3b3b3b
already visible
(see input.form-search in /batrik/css/components/form.css).

However, attaching a reroll.

adci_contributor’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
emma.maria’s picture

If you cannot reproduce an issue, you can always set it to be Closed (cannot reproduce). If you are wrong someone will reopen it :)

emma.maria’s picture

Status: Needs review » Closed (duplicate)
Related issues: +#2281451: Search form enabled in header region does not show the typed text.

I found the issue that duplicates this and was fixed #2281451: Search form enabled in header region does not show the typed text.. Closing this issue.