8000 method isShowing() · Issue #55 · ViHtarb/Tooltip · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

method isShowing() #55

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.

Sign up for GitHub

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

Jump to bottom
Closed
mariooooo opened this issue May 8, 2019 · 13 comments
Closed

method isShowing() #55

mariooooo opened this issue May 8, 2019 · 13 comments

Comments

@mariooooo
Copy link
mariooooo commented May 8, 2019 •
edited
Loading

method isShowing() always false doesn't work in version 0.1.9

tooltip = new Tooltip.Builder(registration_or)
.setText(getResources().getString(R.string.account_registration_instruction))
.setCancelable(true)
.setDismissOnClick(true)
.setTextColor(getResources().getColor(R.color.white))
.setBackgroundColor(getResources().getColor(R.color.mainGreen))
.setTypeface(fontBoldIt)
.setTextSize(18f)
.setCornerRadius(5f)
.setPadding(50f)
.show();
if(tooltip.isShowing()){
System.out.println("TOOLTIP IS SHOWING!!!!!");
}
@mariooooo mariooooo reopened this May 10, 2019
@mariooooo
Copy link
Author

Method isShowing() work fine. The problem is different, I have a button that, when pressed, opens or closes tooltip, but when tooltip is shown and i press the button to close tooltip, first triggered (isCancelable && event.getAction() == MotionEvent.ACTION_OUTSIDE) event then triggered my button click event, then i call isShowing() and always get false. So my tooltip first closes and then opens again instantly. Any idea how to fix this?

public void tooltipBtnClicked() {
        if (tooltip.isShowing()) {
            tooltip.dismiss();
        } else {
            tooltip.show();
        }
    }

@ViHtarb
Copy link
Owner
ViHtarb commented May 10, 2019

Hi. Y 8000 ou can use setCancelable(false). This will disable dismissing the tooltip on outside touch

@mariooooo
Copy link
Author

but I need dismiss when outside, on tooltip and button click

@ViHtarb
Copy link
Owner
ViHtarb commented May 11, 2019

Давай на русском) ты можешь объяснить зачем нужна такая логика? Просто в том ключе, в котором я понимаю тебя - это противоречит идее тултипов

@mariooooo
Copy link
Author

ну у меня два инпута, использовать надо только один, между ними я поставил иконку знака вопроса по нажатию должна выскакивать инструкция, закрываться и открываться снова если надо. я не могу понять почему outside click так работает регистрирует два клика подряд

@ViHtarb
Copy link
Owner
ViHtarb commented May 12, 2019 8000

Кажись я тебя понял. Собери тестовый проект и посмотри там все норм пашет? Я вроде бы это фиксил уже после 0.2.0 версии потому советую юзать 1.0.0-alpha01, хоть она и в альфе, но вроде бы достаточно стабильна, я у себя в проектах юзаю без проблем

@mariooooo
Copy link
Author

попробовал 1.0.0-alpha01, все равно не получилось, для наглядности записал видео https://youtu.be/I_EWR-SsOOM
еще какие-то идеи будут?

@mariooooo
Copy link
Author
mariooooo commented May 13, 2019 •
edited
Loading 
popupWindow.setBackgroundDrawable(new StateListDrawable())
popupWindow.setOutsideTouchable(true);
popupWindow.setFocusable(true);

В общем setFocusable пофиксил все, еще было бы круто если ширину и высоту popupWindow можно было задавать руками

mPopupWindow.setWidth(ViewGroup.LayoutParams.MATCH_PARENT);
mPopupWindow.setHeight(ViewGroup.LayoutParams.WRAP_CONTENT);

и еще самое такое жесткое добавить возможность скролить. Когда можно ждать апдейт примерно?

@ViHtarb
Copy link
Owner
ViHtarb commented May 13, 2019

Да, setFocusable это фиксит) Сорри, я почему-то думал, что запаблишил этот фикс, в локальных изменениях он у меня есть и уже давно и в семпл проекте - такой баги нет.
На счет изменения размеров - в будущем мб сделаю, посмотрим, сейчас для "Simple text Tooltip implementation" есть возможность задавать setMaxWidth.
Начиная с alpha01 присутствует поддержка "хоть какой-то" кастомизации тултипов, так что в целом - можешь сам реализовать почти все что нужно.

@ViHtarb
Copy link
Owner
ViHtarb commented May 13, 2019

За возможность скроллить - пока что хз, опять же, под тултипами подразумевается просто всплывающая подсказка в которой не должно быть много текста, как пример - можно рассматривать тултипы от самих гуглов.

@mariooooo
Copy link
Author

я вот еще пробовал ставить setMaxWidth(MATCH_PARENT) и оно почему-то не работает и на 0.2.0 и на 1.0.0-alpha01 , а еще забыл сказать что если setCancelable(true) то и Focusable(true) и наоборот иначе setCancelable(false) работать не будет если setFocusable(true), еще бы я конечно padding Left, Right, Top, Bottom добавил

@ViHtarb
Copy link
Owner
ViHtarb commented May 13, 2019

за setFocusable в курсе за setMaxWidth - это не одно и тоже что setWidth и он не принимает WRAP_CONTENT и тд

@ViHtarb
Copy link
Owner
ViHtarb commented May 13, 2019

для паддингов - на то я и сделал кастомизацию что бы можно было делать че и как хотите, в будущем реализация простого тултипа будет не final и его тоже можно будет править, но то потом

ViHtarb added a commit that referenced this issue May 14, 2019
@ViHtarb
Updated library to 1.0.0-alpha01-SNAPSHOT
d835815 
Migrated to Java 1.8
Fixed simple Tooltip Builder.setText(int) method
Implemented sets Tooltip setFocusable(isCancelable) issue #55
@ViHtarb ViHtarb closed this as completed May 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ViHtarb @mariooooo
0