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

[RFC] [Ui] Cancel button #4752

Merged
merged 1 commit into from
Apr 12, 2016
Merged

Conversation

lchrusciel
Copy link
Member

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

It was annoying, that it wasn't possible to just go back on create/update form. It is not the prettiest solution, but I wasn't able to find a better one. Also I have not add second parameter as a fallback, because I didn't want to add parameter to this templates. But on the other hand I don't have any good idea, how to solve this.

Before:
zrzut ekranu 2016-04-12 o 08 19 02

After:
zrzut ekranu 2016-04-12 o 08 22 05

@pjedrzejewski pjedrzejewski added the UX Issues and PRs aimed at improving User eXperience. label Apr 11, 2016
@pjedrzejewski pjedrzejewski merged commit a66df62 into Sylius:master Apr 12, 2016
@pjedrzejewski
Copy link
Member

Nice, thanks Łukasz!

@pjedrzejewski
Copy link
Member

Perhaps later we can extract this to separate macro/template to avoid duplication. 👍

@lchrusciel
Copy link
Member Author

👍

@lchrusciel lchrusciel deleted the cancel-button branch April 12, 2016 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UX Issues and PRs aimed at improving User eXperience.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants