Skip to content

Changed me page data editing#20

Open
ZhangYihanNus wants to merge 8 commits intoMaxPoon:masterfrom
ZhangYihanNus:userinfo
Open

Changed me page data editing#20
ZhangYihanNus wants to merge 8 commits intoMaxPoon:masterfrom
ZhangYihanNus:userinfo

Conversation

@ZhangYihanNus
Copy link
Collaborator

@ZhangYihanNus ZhangYihanNus commented Mar 17, 2019

Changed me page data editing method to using a separated page
微信图片_20190318004910
微信图片_20190318004925
微信图片_20190318004932
微信图片_20190318004937

@ZhangYihanNus ZhangYihanNus changed the title Changed me page data editing Changed me page data editing and added message framework Jun 1, 2019
@ZhangYihanNus ZhangYihanNus changed the title Changed me page data editing and added message framework Changed me page data editing Jun 1, 2019
})
},

formSubmit: function (e) { },
Copy link
Owner

Choose a reason for hiding this comment

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

Redundant?

btnDisable: event.detail.value.length === 0
});
},
formSubmit(e) {
Copy link
Owner

Choose a reason for hiding this comment

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

let's use the same format - either functionName: function() or functionName().

@MaxPoon
Copy link
Owner

MaxPoon commented Jun 1, 2019

The screenshot looks good. Let's show a blank field instead of "null" for empty fields.

@zhangxinyi0106
Copy link
Collaborator

Can we set the input fields' names and values to different colours (e.g. black and grey) for users to see more clearly?

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

Successfully merging this pull request may close these issues.

3 participants