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

Try again can't be translated #155

Closed
Alex293 opened this issue Dec 9, 2016 · 5 comments
Closed

Try again can't be translated #155

Alex293 opened this issue Dec 9, 2016 · 5 comments

Comments

@Alex293
Copy link
Contributor

Alex293 commented Dec 9, 2016

Hi,

Working on translating my app I found that there is no way to easily change the title of the Try again button in the ResourceStatusOverlay. There is an option for the title so it may be a good idea to do the same for the button ?

@pcantrell
Copy link
Member

I’d welcome a PR for this. I’m not sure whether it’s preferable to use base internationalization and have strings files for the storyboard, or just set a localized button title in code.

Opinions on that?

@Alex293
Copy link
Contributor Author

Alex293 commented Dec 12, 2016

I feel like ResourceStatusOverlay is supposed to be used as a drop-in composant and should not not be subclassed or customised a lot. So it would be very convenient that Siesta handle internationalisation but It would require then a lot's of work to keep it compatible with a large number of langages. On the other hand providing an optional title property wouldn't be a great effort and would give the user a more complete customisation.

@pcantrell
Copy link
Member

I’d be fine with exposing a retryButton property for the button itself, much like the current errorHeadline. Would that suit your needs?

@Alex293
Copy link
Contributor Author

Alex293 commented Dec 14, 2016

Sure

@pcantrell
Copy link
Member

Go ahead and submit a pull request for this. I’d be happy to accept it!

pcantrell added a commit that referenced this issue Dec 19, 2016
Some small improvements to SiestaUI proposed in #131 and in #155
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

No branches or pull requests

2 participants