eshcherbin / save-the-moment

SPbAU educational Android project
0 stars 0 forks source link

Now EditText with title has only one line #29

Closed egor-bogomolov closed 7 years ago

egor-bogomolov commented 7 years ago

Теперь в MomentEditor при заходе фокус устанавливается на пустой layout, Title состоит из одной строки.

krinkinmu commented 7 years ago

Из того что вы написали следует, что вам нужны оба свойства.

2017-01-07 16:52 GMT+03:00 egor-bogomolov notifications@github.com:

@egor-bogomolov commented on this pull request.

In app/src/main/res/layout/activity_momenteditor.xml https://github.com/eshcherbin/save-the-moment/pull/29:

     <EditText

android:gravity="center" android:hint="@string/edittext_momenteditor_title_text" android:textSize="25sp" android:layout_width="match_parent" android:layout_height="wrap_content" android:id="@+id/edittext_momenteditor_title"

  • android:layout_weight="1" />
  • android:layout_weight="1"
  • android:inputType="text"/>

Опции, явно указывающие, что текст - однострочный (maxLines = 1, пришедший на смену singleLIne = true), однострочным делают не текст, а поле ввода. То есть при нажатии на Enter строка переносится, просто поле ввода не раздвигается. Изменение же inputType с none на text как раз позволяет одновременно и добавить однострочность, и изменить функциональность Enter на смену фокуса на следующее поле ввода.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/eshcherbin/save-the-moment/pull/29, or mute the thread https://github.com/notifications/unsubscribe-auth/ABH4c8QxgFWMd6522M3gcXoCfNjJAxAPks5rP5iLgaJpZM4LVc44 .

egor-bogomolov commented 7 years ago

Да, нужны.

egor-bogomolov commented 7 years ago

Проблема в том, что это не отражено в PR?

krinkinmu commented 7 years ago

У вас используется одна опция из нужных 2, очевидно, что это странно.

On Sat, Jan 7, 2017 at 5:00 PM, egor-bogomolov notifications@github.com wrote:

Проблема в том, что это не отражено в PR?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/eshcherbin/save-the-moment/pull/29#issuecomment-271085438, or mute the thread https://github.com/notifications/unsubscribe-auth/ABH4c-rvDuECMp2QucOJJe9KIUdVCfBrks5rP5qZgaJpZM4LVc44 .

egor-bogomolov commented 7 years ago

Я понял, сейчас переделаю