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

🐛 fix: should parse textarea value and placeholder #161

Merged
merged 3 commits into from
Sep 14, 2022
Merged

Conversation

MadCcc
Copy link
Member

@MadCcc MadCcc commented Sep 14, 2022

fix #142

@MadCcc MadCcc changed the title fix(pencil): should parse textarea value and placeholder fix: should parse textarea value and placeholder Sep 14, 2022
const { value, placeholder } = node as HTMLInputElement;
if (!value && !placeholder) return;
const { value, placeholder } = node;
if (!value && !placeholder) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里为啥要加括号?

if (value) {
console.log(value)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个去掉?

@arvinxx
Copy link
Collaborator

arvinxx commented Sep 14, 2022

Commit 改成 gitmoji commit ? 这样合并以后会自动发包 https://www.yuque.com/arvinxx-fe/workflow/gitmoji-commit-workflow

@MadCcc MadCcc changed the title fix: should parse textarea value and placeholder 🐛fix: should parse textarea value and placeholder Sep 14, 2022
@MadCcc MadCcc changed the title 🐛fix: should parse textarea value and placeholder 🐛 fix: should parse textarea value and placeholder Sep 14, 2022
@arvinxx arvinxx merged commit 06d70ac into beta Sep 14, 2022
@arvinxx arvinxx deleted the fix/textarea branch September 14, 2022 08:39
github-actions bot pushed a commit that referenced this pull request Sep 14, 2022
# [1.0.0-beta.15](v1.0.0-beta.14...v1.0.0-beta.15) (2022-09-14)

### 🐛 Bug Fixes

* should parse textarea value and placeholder ([06d70ac](06d70ac)), closes [#161](#161)
@github-actions
Copy link

🎉 This PR is included in version 1.0.0-beta.15 🎉

The release is available on:

Your semantic-release bot 📦🚀

github-actions bot pushed a commit that referenced this pull request Feb 9, 2023
# [1.0.0](v0.4.7...v1.0.0) (2023-02-09)

### ♻ Code Refactoring

* rename the symbol function ([09d0de9](09d0de9))

### ✨ Features

* input support overflow ([0ff7b55](0ff7b55)), closes [#177](#177)
* support almost full css transform ([f7bb06d](f7bb06d)), closes [#148](#148)
* support overflow element ([f4b017b](f4b017b)), closes [#178](#178)
* support rad angle line-gradient ([9860090](9860090)), closes [#96](#96)
* support rotate & clip-path ([35f6628](35f6628)), closes [#180](#180)
* 优化抓取 base64 图片方法 ([07dd128](07dd128))
* 提供自定义 fetch 方法的能力 ([28ed276](28ed276))
* 新增固定宽高的能力 ([f2df2e5](f2df2e5))

### 🐛 Bug Fixes

* adjust text and svg resizing constraint ([9fc8970](9fc8970))
* fix absolute position margin parsing ([120ba39](120ba39))
* fix overflow hidden node use clipping mask ([defb941](defb941))
* fix overflow hidden node use clipping mask ([055f25d](055f25d))
* fix password input text ([c8ff928](c8ff928))
* fix placeholder parsing without css rules ([505a390](505a390))
* fix position bug due to sketch text align parsing error ([1dce98f](1dce98f)), closes [#51](#51)
* fix pseudo elt style issue and add test ([5c727a3](5c727a3))
* fix resizingConstraint func and update test ([c39f843](c39f843))
* gradient parse in safari ([aa374e5](aa374e5)), closes [#185](#185)
* parse linear-gradient ([90f3ed8](90f3ed8)), closes [#184](#184)
* set default resizingConstraint to left and top ([f897275](f897275))
* should parse textarea value and placeholder ([06d70ac](06d70ac)), closes [#161](#161)
* single child overflow should not be mask ([1acab12](1acab12)), closes [#181](#181)
* try to fix babel problem ([398f825](398f825))
* 修正 textStyle 缺失导致修改文本造成报错的问题 ([1a44244](1a44244))

### 💥 BREAKING CHANGES

* rename `nodeToSketchSymbol` to nodeToSymbol
@github-actions
Copy link

github-actions bot commented Feb 9, 2023

🎉 This PR is included in version 1.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

None yet

2 participants