jimpanzer15 мая 2013 в 13:02

Пишем свой Watermark TextBox и PasswordBox для Win8/RT, Windows Phone

7 мин

Программирование * Windows Phone *

Из песочницы

Комментарии 7

FanAs 15 мая 2013 в 18:27

Прикладывайте пожалуйста скриншоты, хотя бы из эмулятора или редактора XAML-кода.

jimpanzer 15 мая 2013 в 19:07

Обязательно выкину завтра. Спасибо за фидбек.

DeeKey 15 мая 2013 в 23:08

Во время написания статьи наткнулся на Dependency Property Generator. Надеюсь это тоже будет экономить время.

propdp <Tab> <Tab> как-то проще — не надо никуда из студии переключаться.

GrigoryPerepechko 18 мая 2013 в 08:42

О, спасибо. Я раньше думал что это решарпер.

GrigoryPerepechko 18 мая 2013 в 08:55

Идея хорошая, но качество кода очень печальное. Проблемы все мелкие, но режут глаз. Форматирование, названия переменных/контролов/методов, всякие "" вместо каноничного string.Emtpy, да даже Color.FromArgb(255, 255, 255, 255) вместо которого можно взять Colors.White. Вперемешку идут private static и public instance. Методы вперемешку со свойствами. Джавадоки а-ля

/// summary
/// TextSize
/// /summary

Установление свойств а-ля Margin/Padding/BorderThickness/Background в их значения по умолчанию (ну зачем это делать),
Установление FontSize в магические 14.667 — вы долго вычисляли это значение? Ну почему не использовать системные ресурсы? Там все это есть.

Конструкция


  if (val == null)
            {
                controll.tb.Text = "";
                return;
            }
            controll.tb.Text = val;

вообще вогнала меня в ступор. Говнокод в стиле индусов микрософта.

jimpanzer 20 мая 2013 в 08:56

Спасибо за Ваш комментарий — он принят к сведению. Если есть конкретные предложения по улучшению поста/кода с удовольствием прочитаю их в личных сообщениях

ps. По поводу Вашего удивления «магической» цифрой 14,667 — все просто. Достаточно почитать про Guidelines for fonts (Windows Store apps).
Если достовно:

Use 11 pt Segoe UI Semilight for most text.

11 pt = 14.667 px (1pt==(96/72) px)

ps2. использую Blend и цифры не придумываю.

GrigoryPerepechko 20 мая 2013 в 09:23

Какие ещё вам предложения нужны? Я уже написал список, поправьте хоть это. Или я должен за вас переписать весь код?

А что касается 14.667. Я ещё раз говорю — есть системные стили, не нужно цифр.
Вместо

FontSize="14.667"

напишите

FontSize="{StaticResource ControlContentThemeFontSize}

Зарегистрируйтесь на Хабре, чтобы оставить комментарий