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

Flarum ColorInput textbox issues #3858

Closed
Ralkage opened this issue Jul 22, 2023 · 0 comments · Fixed by #3919
Closed

Flarum ColorInput textbox issues #3858

Ralkage opened this issue Jul 22, 2023 · 0 comments · Fixed by #3919
Labels

Comments

@Ralkage
Copy link
Contributor

Ralkage commented Jul 22, 2023

Current Behavior

The ColorInput textboxes in Flarum, more specifically in the Admin CP under "Appearance" and new/edit groups modal, are automatically completing the hex you put in even when you don't want it to. It seems to only happen when you input consecutive lowercase letters and numbers after the pound symbol. Also, when you try to backspace to input the correct hex, it'll keep adding back whatever letter or number sequence you attempted to delete during the process.

Steps to Reproduce

  1. Go into the Admin CP.
  2. Click on "Appearance".
  3. Under the "Colors" section, clear the first input textbox.
  4. Start typing a pound symbol followed by a sequence of numbers or lowercase letters such as #333 or #aaa.
  5. Hit backspace while at the end of the text input within the textbox.

Expected Behavior

The ColorInput textboxes in Flarum should not automatically complete the hex input text, especially when you want to add hex code that is non-sequential in letters or numbers.

Screenshots

2023-07-21_21-08-50.mp4

Environment

  • Flarum version: 1.8.1
  • Website URL: localhost
  • Webserver: Apache
  • Hosting environment: Local machine
  • PHP version: 8.1.10
  • Browser: Microsoft Edge - Version 114.0.1823.82 (Official build) (64-bit)

Output of php flarum info

Flarum core: 1.8.1
PHP version: 8.1.10
MySQL version: 8.0.30
Loaded extensions: Core, bcmath, calendar, ctype, date, filter, hash, iconv, jso
n, SPL, pcre, readline, Reflection, session, standard, mysqlnd, tokenizer, zip,
zlib, libxml, dom, PDO, openssl, SimpleXML, xml, xmlreader, xmlwriter, curl, fil
einfo, gd, intl, mbstring, exif, mysqli, Phar, pdo_mysql, xsl
+----------------------+---------+--------+
| Flarum Extensions    |         |        |
+----------------------+---------+--------+
| ID                   | Version | Commit |
+----------------------+---------+--------+
| flarum-flags         | v1.8.0  |        |
| flarum-approval      | v1.8.0  |        |
| flarum-tags          | v1.8.0  |        |
| flarum-suspend       | v1.8.0  |        |
| flarum-subscriptions | v1.8.0  |        |
| flarum-sticky        | v1.8.0  |        |
| flarum-statistics    | v1.8.0  |        |
| flarum-mentions      | v1.8.2  |        |
| flarum-markdown      | v1.8.0  |        |
| flarum-lock          | v1.8.0  |        |
| flarum-likes         | v1.8.0  |        |
| flarum-lang-english  | v1.8.0  |        |
| flarum-emoji         | v1.8.0  |        |
| flarum-bbcode        | v1.8.0  |        |
+----------------------+---------+--------+
Base URL: https://flarumdev.test
Installation path: C:\laragon\www\flarumdev
Queue driver: sync
Session driver: file
Mail driver: mail
Debug mode: off

Possible Solution

N/A

Additional Context

Wasn't sure how to word the title, so if it doesn't make sense, feel free to edit it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant