soarcn/UndoBar

Using the old UndoBar

tasomaniac opened this issue · 3 comments

When I am trying to show a new UndoBar, you are using the old one and customize it. This makes sense. But there is a problem. If the bar is a just a message bar you are hiding the buttons and then if it is a UndoBar you are not showing them again.

You shoul add to that if statement. I wish I could send you a pull request but I am new here. :)

            button.setVisibility(View.VISIBLE);
            findViewById(id.undobar_divider).setVisibility(View.VISIBLE);

Thanks for this, Message Bar style was introduced at quite late phase, and I haven't use this in my own project, I prefer use crouton for better message queue(which is not UndoBar focus to), this has been fixed in latest pull. :)

I was using Crouton but having normal messages at top and error, retry and
undo messages at the bottom seemed to me interesting. It is not very user
friendly I guess. I have switched from Crouton to this and I am very happy
with it. It is not a bug at all. Thank you.

On Sat, May 25, 2013 at 2:25 PM, Liao Kai notifications@github.com wrote:

Thanks for this, Message Bar style was introduced at quite late phase, and
I haven't use this in my own project, I prefer use crouton for better
message queue(which is not UndoBar focus to), this has been fixed in latest
pull. :)


Reply to this email directly or view it on GitHubhttps://github.com//issues/2#issuecomment-18445636
.

std

http://tasomaniac.com http://card.ly/taso
http://home.ku.edu.tr/~sdane

:) feel free to use this, if there are any defects or suggestions, put them to here~