Search
Write a publication
Pull to refresh

Comments 7

Прикладывайте пожалуйста скриншоты, хотя бы из эмулятора или редактора XAML-кода.
Обязательно выкину завтра. Спасибо за фидбек.
Во время написания статьи наткнулся на Dependency Property Generator. Надеюсь это тоже будет экономить время.

propdp <Tab> <Tab> как-то проще — не надо никуда из студии переключаться.
Идея хорошая, но качество кода очень печальное. Проблемы все мелкие, но режут глаз. Форматирование, названия переменных/контролов/методов, всякие "" вместо каноничного 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;
вообще вогнала меня в ступор. Говнокод в стиле индусов микрософта.
Спасибо за Ваш комментарий — он принят к сведению. Если есть конкретные предложения по улучшению поста/кода с удовольствием прочитаю их в личных сообщениях

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 и цифры не придумываю.
Какие ещё вам предложения нужны? Я уже написал список, поправьте хоть это. Или я должен за вас переписать весь код?

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

напишите
FontSize="{StaticResource ControlContentThemeFontSize}

Sign up to leave a comment.

Articles