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

[Web][Form] - fixed unclickable upload buttons issue 3263 #3314

Conversation

nakashu
Copy link
Contributor

@nakashu nakashu commented Sep 21, 2015

Q A
Bug fix? yes
Enhancement? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets #3263
License MIT

css/ hmtl changes

Styled upload button area is now fully clickable
Changes where also made in product image prototype html, for it to be consistent/working
Will also fix bugs with translated messages for upload button, as there is no need for the fixed width, that was present before

javascript - enhancement

Filename of file that would be uploaded is shown, next to upload button.
Works with multiple file upload fields also if browser supports HTML5 File API

clickable areas on upload buttons, fixed by replacing <span> with <label for="id">
removed the fixed width in css that would also complicate translation strings
change made also in image prototype html inside [Product] for consistency
…t to the upload button

After choosing a file to upload, its name will be shown, next to the upload button to indicate what will be uploaded.
Work with multiple file upload field also, but only when HTML5 Files API is supported (ie10+)
@nakashu
Copy link
Contributor Author

nakashu commented Sep 23, 2015

can anyone explain why is the test failing?
I cannot find how it is related to the changes made in the commits.

@pamil
Copy link
Contributor

pamil commented Sep 23, 2015

@nakashu no fear, I'll take care of these bugs today :)

@nakashu
Copy link
Contributor Author

nakashu commented Sep 28, 2015

bump for merge ?

pjedrzejewski pushed a commit that referenced this pull request Sep 28, 2015
…ns_issue_3263

[Web][Form] - fixed unclickable upload buttons issue 3263
@pjedrzejewski pjedrzejewski merged commit 1408fa6 into Sylius:master Sep 28, 2015
@pjedrzejewski
Copy link
Member

Thank you @nakashu! Great work!

@nakashu nakashu deleted the bugfix/unclickable_upload_buttons_issue_3263 branch February 16, 2016 22:10
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