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.

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

Add field text with range #134

Draft
wants to merge 5 commits into
base: feature/v2_carbonfay_round1
Choose a base branch
from
Draft

Add field text with range #134

wants to merge 5 commits into from

Conversation

olddeda
Copy link
Contributor

@olddeda olddeda commented Jun 28, 2019

Before use need approve #133 and #132

@olddeda
Add field text with range
74a8a23 
Before use need approve #133 and #132
@evgeny-boger
Copy link
Member

а про какой issue это?

@olddeda
Copy link
Contributor Author

olddeda commented Jul 3, 2019

а про какой issue это?

https://github.com/contactless/homeui/issues/118

evgeny-boger
evgeny-boger requested changes Aug 30, 2019
View reviewed changes
Copy link
Member

@evgeny-boger evgeny-boger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

что-то это выглядит максимально страшно:

image

хочется это как-то в одну линию наверное сделать. Ну или как-то ещё, но сейчас - очень страшно.

@evgeny-boger
Copy link
Member

всё максимально плохо. При изменении числа в поле не бегает ползунок. В MQTT не отправляется значение. Клавиатурой поменять нельзя. При нажатии на поле зачем-то копируется значение в буфер обмена. Но при этом не текущее, т.к. поле не меняет значение. И при нажатии на кнопки вверх-вниз тоже вылезает эта всплывающая фигня про копирование значения!

@olddeda
Copy link
Contributor Author

olddeda commented Sep 11, 2019

всё максимально плохо. При изменении числа в поле не бегает ползунок. В MQTT не отправляется значение. Клавиатурой поменять нельзя. При нажатии на поле зачем-то копируется значение в буфер обмена. Но при этом не текущее, т.к. поле не меняет значение. И при нажатии на кнопки вверх-вниз тоже вылезает эта всплывающая фигня про копирование значения!

Before use need approve #133 and #132, задача зависила от них

@olddeda
Merge commit '8766eed36286d7e7eecdbfc6920542b1249c7272' into feature/…
9ac0ff5 
…118-add-field-for-range

# Conflicts:
#	app/scripts/directives/displaycell.html
#	app/scripts/directives/textcell.html
@evgeny-boger
Copy link
Member

так, странное поведение. Захожу в это поле, пишу туда на клавиатуре число, нажимаю enter, попадаю в history. Если не захожу в поле и нажимаю энтер, то не попадаю в хистори. Странный баг.

@olddeda
Copy link
Contributor Author

olddeda commented Sep 11, 2019

так, странное поведение. Захожу в это поле, пишу туда на клавиатуре число, нажимаю enter, попадаю в history. Если не захожу в поле и нажимаю энтер, то не попадаю в хистори. Странный баг.

поправил

@evgeny-boger
Copy link
Member

Стили съехали:
image

@sikmir sikmir marked this pull request as draft July 30, 2024 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@evgeny-boger evgeny-boger evgeny-boger requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@olddeda @evgeny-boger