Skip to content

style: input disabled style#2861

Merged
Ricbet merged 1 commit intomainfrom
style/input-disabled
Jul 4, 2023
Merged

style: input disabled style#2861
Ricbet merged 1 commit intomainfrom
style/input-disabled

Conversation

@Ricbet
Copy link
Copy Markdown
Member

@Ricbet Ricbet commented Jul 3, 2023

Types

  • 💄 Style Changes

Background or solution

🤖 Generated by Copilot at dba44df

  • Add a modifier class .input-disabled to the input component style and apply a not-allowed cursor to the input element (link)

之前 input 的 disabled 样式是不生效的,现调整成正确的方式

Changelog

🤖 Generated by Copilot at dba44df

Refactored the input component style to use a modifier class for the disabled state and added a not-allowed cursor. This improves the component's usability and alignment with other components.

@Ricbet Ricbet self-assigned this Jul 3, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 3, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (472d189) 57.63% compared to head (dba44df) 57.63%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2861   +/-   ##
=======================================
  Coverage   57.63%   57.63%           
=======================================
  Files        1336     1336           
  Lines       83983    83983           
  Branches    17462    17462           
=======================================
+ Hits        48402    48406    +4     
+ Misses      32338    32334    -4     
  Partials     3243     3243           
Flag Coverage Δ
jsdom 52.66% <ø> (+<0.01%) ⬆️
node 16.78% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Ricbet Ricbet merged commit 0a02fdd into main Jul 4, 2023
@Ricbet Ricbet deleted the style/input-disabled branch July 4, 2023 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants