fox :: from/btimofeev

Re: Mobile IDEC client

btimofeev — 2017-10-04 20:46:11

btimofeev -> vit01

Вот кусок логката с эмулятора 4.2 (еле скопировал, приложение постоянно рестартится в цикле и логкат заново обновляется):

10-04 16:41:13.172 4868-4868/? E/ACRA: ACRA caught a RuntimeException for vit01.idecmobile
java.lang.RuntimeException: Unable to get provider vit01.idecmobile.FechoFilesProvider: java.lang.ClassNotFoundException: Didn't find class "vit01.idecmobile.FechoFilesProvider" on path: /data/app/vit01.idecmobile-1.apk
at android.app.ActivityThread.installProvider(ActivityThread.java:4822)
at android.app.ActivityThread.installContentProviders(ActivityThread.java:4432)
at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4372)
at android.app.ActivityThread.access$1300(ActivityThread.java:141)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1294)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:137)
at android.app.ActivityThread.main(ActivityThread.java:5041)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:511)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:793)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:560)
at dalvik.system.NativeStart.main(Native Method)
Caused by: java.lang.ClassNotFoundException: Didn't find class "vit01.idecmobile.FechoFilesProvider" on path: /data/app/vit01.idecmobile-1.apk
at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:65)
at java.lang.ClassLoader.loadClass(ClassLoader.java:501)
at java.lang.ClassLoader.loadClass(ClassLoader.java:461)
at android.app.ActivityThread.installProvider(ActivityThread.java:4807)
at android.app.ActivityThread.installContentProviders(ActivityThread.java:4432)
at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4372)
at android.app.ActivityThread.access$1300(ActivityThread.java:141)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1294)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:137)
at android.app.ActivityThread.main(ActivityThread.java:5041)
at java.lang.reflect.Method.invokeNative(Native Method)
at java.lang.reflect.Method.invoke(Method.java:511)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:793)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:560)
at dalvik.system.NativeStart.main(Native Method)
10-04 16:41:13.172 4868-4868/vit01.idecmobile I/dalvikvm: Could not find method android.os.StatFs.getBlockSizeLong, referenced from method org.acra.collector.MemoryInfoCollector.getAvailableInternalMemorySize
10-04 16:41:13.172 4868-4868/vit01.idecmobile W/dalvikvm: VFY: unable to resolve virtual method 1465: Landroid/os/StatFs;.getBlockSizeLong ()J
10-04 16:41:13.172 4868-4868/vit01.idecmobile D/dalvikvm: VFY: replacing opcode 0x6e at 0x0013
10-04 16:41:13.172 4868-4868/vit01.idecmobile I/dalvikvm: Could not find method android.os.StatFs.getBlockSizeLong, referenced from method org.acra.collector.MemoryInfoCollector.getTotalInternalMemorySize
10-04 16:41:13.172 4868-4868/vit01.idecmobile W/dalvikvm: VFY: unable to resolve virtual method 1465: Landroid/os/StatFs;.getBlockSizeLong ()J
10-04 16:41:13.172 4868-4868/vit01.idecmobile D/dalvikvm: VFY: replacing opcode 0x6e at 0x0013
10-04 16:41:13.172 4868-4868/vit01.idecmobile I/dalvikvm: Could not find method android.media.MediaCodecList., referenced from method org.acra.collector.MediaCodecListCollector.collectMediaCodecList
10-04 16:41:13.172 4868-4868/vit01.idecmobile W/dalvikvm: VFY: unable to resolve direct method 1051: Landroid/media/MediaCodecList;. (I)V
10-04 16:41:13.172 4868-4868/vit01.idecmobile D/dalvikvm: VFY: replacing opcode 0x70 at 0x001e

Re: idec.el

btimofeev — 2017-09-29 21:31:19

btimofeev -> vit01

vit01> А что за тема оформления у твоего емакса?

Могу ошибаться, но похоже что это monokai.

Re: Фэхи

btimofeev — 2017-08-03 14:01:58

btimofeev -> vit01

vit01> У меня на станции стоят строгие регулярки, по которым имя файла разрешается только в lowercase (как имя эхи). Вроде бы, мы именно так по стандарту и договаривались, не?

А чем плохи заглавные буквы в именах файлов?

Re: Фэхи

btimofeev — 2017-07-31 21:21:45

btimofeev -> Andrew Lobanov

AL> В таверне на данный момент есть следующий фэхи:

А как их получать? В цезии нужно фэхи прописывать в конфиге? По умолчанию он мне пишет новых файлов не найдено.

Re: Фэхи

btimofeev — 2017-07-31 21:28:25

btimofeev -> btimofeev

btimofeev> А как их получать? В цезии нужно фэхи прописывать в конфиге? По умолчанию он мне пишет новых файлов не найдено.

Разобрался. Нужно в конфиг добавить "fecho имя_фэхи". В readme бы добавить это.

Re: Android клиент

btimofeev — 2017-07-22 13:46:28

btimofeev -> vit01

jmaks>> Реквестирую СОХРАНЕНИЕ черновика сообщения

vit01> Он и так должен сохраняться, если ты кнопку "назад" нажимаешь.

Похоже нужно ещё и в onPause сохранять.

Re: Мысли о стандартах

btimofeev — 2017-07-20 16:52:47

btimofeev -> Andrew Lobanov

AL> Идея личной переписки мёртворождённая. Потому что без шифрования она никому кроме меня не нужна, а с шифрованием мы теряем простоту технологии.

Мне теперь тоже кажется, что не обязательно вводить шифрование в протокол, ведь pgp это тот же текст, так что для протокола не должно быть никакой разницы будет ли передаваться текст сообщения в открытом или зашифрованном виде. Кому нужно шифрование, тот сам добавит в клиенты подобный функционал (ну или хотя бы открытие во внешнем приложении). Простота технологии важнее.

Re: idec mobile

btimofeev — 2017-06-27 11:24:47

btimofeev -> vit01

vit01> 2. Поддержка русских букв в урлах (см. ii://STkMhQ2eSHnTPQqAynts)

У меня по этой ссылке открывается список сообщений в develop, а не само сообщение. Но я ещё не обновлялся.

Re: idec mobile

btimofeev — 2017-06-27 16:25:31

btimofeev -> Andrew Lobanov

AL> А вот колёсико я так и не увидел =(

У меня оно вот такое было http://imgur.com/a/HpZy9

Кстати, на скриншоте оно выглядит не особо и большим. Сейчас обновлюсь на последние коммиты.

Ещё в это окно нужно scroll view добавить: в горизонтальном режиме при наличии ошибки нельзя кнопочку нажать, она не видна за экраном.

msgto

btimofeev — 2017-06-27 16:32:39

btimofeev -> All

В стандарте описано: "msgto - пользователь, которому предназначено сообщение (либо All)".

А что если добавить возможность указывать нескольких пользователей, к примеру, через запятую. На случай, если хочешь, что бы на твоё сообщение обратили внимание определенные пользователи (и что бы сообщение попало в карбонку к ним всем). Кто что думает?

Re: idec mobile

btimofeev — 2017-06-26 20:36:52

btimofeev -> vit01

vit01> И ещё, для всех пользователей клиента я подготовил небольшой сюрприз, о котором здесь не написал.

Какое-то все гигантское, и шрифт, и прогресс-колесо))

Re: idec mobile

btimofeev — 2017-06-26 20:19:03

btimofeev -> vit01

vit01> SDK, кстати, можно брать даже самый новый (но build-tools обязаны быть 23 версии для 32-битных систем).

Не обязательно, я себе с 25 версией собираю, нужно просто в файле app/build.gradle номер версии поменять.

Re: idec mobile

btimofeev — 2017-06-26 21:15:56

btimofeev -> vit01

vit01> Так понятно, что поменять можно. Специально указал, что для 32-битных систем. Или Гугл одумался и выпустил 25 build-tools на старое железо?

Да, ты прав, я про эту проблему и забыл уже.

Re: idec mobile

btimofeev — 2017-06-15 22:28:04

btimofeev -> vit01

vit01> Это не баг, это фича. Я специально захардкодил так, чтобы в результатах поиска и в непрочитанных всегда открывалось само сообщение независимо от настроек. Это сделано в целях экономии времени.

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

Re: idec mobile

btimofeev — 2017-06-15 22:35:18

btimofeev -> vit01

vit01> А ещё в 7 андроиде (да и в 6 тоже) можно менять масштаб изображения на экране в системных настройках.

Не нашел такого у себя в 6-ке, видимо в стандартной прошивке этого нет.

Re: Мысли о стандартах

btimofeev — 2017-06-15 17:58:35

btimofeev -> vit01

AL>> Подписался на фэху и все файлы, попадающие туда, к тебе ссыпятся автоматом. То есть они гейтуются и это в них самое вкусное. В отличии от фреков, мне видится большим преимуществом, что файлы сами будут приходить к пользователям.

vit01> А вот это очень плохо. Когда у меня появятся фэхи, то поставлю квоту на фетчер не более 100 мб в день.
vit01> На клиентах вообще сделаю скачивание исключительно по запросу (обновляться автоматически будут только индексы, а файлы пользователь будет качать сам).

Так я и не понял: один и тот же файл будет храниться на всех серверах? Я думал что только индексы будут гейтоваться, а файл останется на одном сервере. Да и на клиенте мне так же не хотелось бы автоматом выкачивать все файлы.

Re: idec mobile

btimofeev — 2017-06-15 17:17:03

btimofeev -> vit01

vit01> Из новых фич, как вы уже догадались, у нас планшетный режим.

К сожалению потестить не смогу, т.к. нет планшета, но хотелось бы.

vit01> 1. последнее прочитанное (в обычных эхах) выделяется цветом
vit01> 2. включена автопрокрутка до последнего прочитанного

Класс, стало гораздо информативнее.

vit01> 4. В непрочитанных сообщениях и в результатах поиска теперь можно выходить в список сообщений для навигации, как и в нормальных эхах.

Небольшой баг: если отключить опцию "Открывать последнее прочитанное", то в вышеназванных разделах вместо списка открывается сообщение. А после нажатия кнопки "назад" открывается список.

Re: Мысли о стандартах

btimofeev — 2017-06-15 16:54:06

btimofeev -> Andrew Lobanov

AL> Пользователи будут загружать. Удалять не смогут.

В обычных эхах сообщения не удаляемые, имеется черный список. А здесь как? Если сисоп удалит файл, он будет добавлен в черный список? Или клиент при заходе в файлэху будет каждый раз загружать весь список файлов?

Re: Мысли о стандартах

btimofeev — 2017-06-15 15:43:53

btimofeev -> vit01

vit01> В имени файла только английский алфавит, никаких спецсимволов и пробелов.

Хотя бы цифры, нижнее подчеркивание (или дефис, для разделения слов) и точку (для расширения файла) надо бы оставить.

Re: Мысли о стандартах

btimofeev — 2017-06-15 15:38:32

btimofeev -> Andrew Lobanov

А в чем файлэхи будут кардинально отличается от текущего списка файлов? Правильно ли я понимаю, что:

1. Файлы смогут загружать пользователи? Если так, то тут и вправду нужно вводить ограничения. Ты же не обрадуешься, если я к тебе на сервер залью коллекцию своих любимых фильмов. И если это так, то сможет ли пользователь потом удалить файл?

2. Файлэх может быть несколько? То есть можно будет разбить все файлы на категории: софт, музыка и т.п.?

3. Будут ли они гейтоваться между станциями? То есть смогу ли сидя на одной станции получить список файлов другой станции и скачать что-нибудь?

Re: Мысли о стандартах

btimofeev — 2017-06-14 12:49:56

btimofeev -> Andrew Lobanov

AL> Или привлекать PGP и монстрячить способ распросстранения открытых ключей или доверять всем сисопам сети личную переписку, отдавая поинтам только их сообщения.

По-моему лучше первый вариант. Так сделано во многих email и jabber клиентах. Обменялись открытыми ключами, а потом в эхе выбираешь для кого шифруешь сообщения и они отправляются шифрованные, а полученные клиент соответственно расшифровывает твоим приватным ключем.

Re: idec mobile

btimofeev — 2017-06-12 19:29:16

btimofeev -> vit01

vit01> Но пусть будет хотя бы скриншот: https://ii-net.tk/screens/Screenshot_2017-06-12-22-56-49.png

По-моему, выглядит отлично. И наверняка удобно будет.

Re: iing

btimofeev — 2017-05-23 08:00:08

btimofeev -> Andrew Lobanov

AL> Куда катимся. Терминус ШГ называть =)

Мне тоже что-то твой скриншот глаза режет, хотя я сам всегда в терминале терминус использую. Это у тебя кажется terminus bold, наверное поэтому мне не привычно.

Re: Tavern

btimofeev — 2017-05-22 15:22:23

btimofeev -> Andrew Lobanov

AL> WASD для листания сообщения и перелистывания на следующее/предыдущее.
AL> Что скажете?

И на hjkl продублируй, для вимеров)

Re: idec mobile

btimofeev — 2017-05-06 18:54:11

btimofeev -> vit01

vit01> Когда у пользователя рухнуло приложение, то он может открыть логи и скинуть мне трейс.

Это пользователю придется ставить adb на комп, искать в огромном логкате стектрейс. Либо если телефон root'ован, можно логкат просмотреть прямо на телефоне, что попроще, но все равно мало кто таким будет заниматься.

vit01> Очень удобно сделано в F-Droid и NewPipe. Если приложение падает, то открывается Email-клиент со всей нужной информацией, и багрепорт сразу можно отправить на почту разработчикам.

Это совсем другая штука, они не распространяют дебаг версию, у них внутри либы которые занимаются сбором стектрейсов. Типа Firebase Crash, который очень удобен, нужно лишь добавить в gradle.build несколько строк и у тебя в веб интерфейсе будут отображаться все падения со стектрейсами. Но эта штука проприетарная, в свободную программу такое тащить, наверное, не стоит. Есть свободный аналог, Acra, кажется называется, но ее я не пробовал - там вроде нужен собственный сервер.

Re: Странно

btimofeev — 2017-05-06 18:53:36

btimofeev -> vit01

Peter> При заборе сообщений появляется окно отладки, в котором вывод фетчера. После того, как забор завершился, окно висит еще около секунды. В этот момент я судорожно начинаю давить кнопку <. Он говорит мне что дебаг закрывать нехорошо.

Я бы тоже окно дебага убрал бы. Тоже периодически надоедает, да ещё и не понятно что оно там выводит и зачем мне это. Лучше писать эту инфу в файл, а сообщения загружать в фоновом сервисе. Человек при ошибке тебе и файл отправит.

Re: idec mobile

btimofeev — 2017-05-06 10:01:52

btimofeev -> vit01

vit01> Если стек-трейсы будут оставаться прежними (то есть вывод adb страдать не будет), то тогда вполне можно и перейти на релиз-сборки.

А у тебя будет две версии в этом случае собираться. Себе будешь ставить дебаг версию, пользователям релиз. Просто пользователям незачем тратить лишний трафик и ресурсы телефона.

У меня в патчере, например, команда gradle build собирает сразу восемь apk, релиз и дебаг версии для четырех магазинов приложений (с немного измененными настройками, а для Google Play и с измененными исходниками). Там все довольно гибко можно настроить.

Re: Странно

btimofeev — 2017-05-06 07:38:55

btimofeev -> Andrew Lobanov

AL> Я всего лишь нажал на кнопку "Отправить". Откуда он взял столько странных сообщений?

Жмешь кнопку ответить с цитатой, ничего не пишешь, выходишь назад, клиент сохраняет сообщение в черновиках. Так несколько раз. Я обычно перед отправкой черновики просматриваю.

Может при выходе из редактирования спрашивать сохранять ли сообщение в черновике? Или определять, что поинт ничего не написал.

Re: idec mobile

btimofeev — 2017-05-06 07:15:21

btimofeev -> vit01

vit01> Принципиального различия между ними не вижу, кроме возможности публиковать в маркетах.

Ещё релиз меньше по размеру, т.к. не содержит отладочной информации (а в дополнение можно ещё и Proguard'ом сжать). Ну и как ты уже написал ключ через год не испортится.

Re: idec mobile

btimofeev — 2017-05-05 21:02:15

btimofeev -> vit01

AL> Похоже, я постоянно ставил какую-то древнюю версию.

vit01, может добавить в клиент версионирование? Да и не плохо бы распространять релиз версию, ты же вроде до сих пор дебаг ключом подписываешь? (Только при этом всем придётся удалить текущую, с бэкапом настроек).

Re: idec mobile

btimofeev — 2017-05-04 10:50:07

btimofeev -> vit01

vit01> // Зря ты это в ii.14 написал, ибо новая фича есть только в git, да и вообще это должно было быть сюрпризом :)

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

Re: idec mobile

btimofeev — 2017-05-03 23:56:15

btimofeev -> vit01

Ещё нельзя удалить название эхи из расширенного поиска: удаляю, закрываю, открываю снова и та же эха вписана снова.

Re: idec mobile

btimofeev — 2017-05-03 23:50:36

btimofeev -> vit01

Парочка багов в VIP:

Фон у bottom sheet в тёмных темах белый и текст на нём не виден, бэкграунд наверное нужно из основных цветов темы брать, либо задать отдельную переменную для него.

Если я ничего не ввожу в поиск и жму кнопку "начать поиск", то приложение падает.

А если я что-то ввожу в поиск, то кнопка "начать поиск" куда-то пропадает из расширенных настроек, приходится закрывать расширенные и нажимать enter на клавиатуре в верхнем поле ввода.

Саму кнопочку поиска можно бы заменить на floating action button прикрепленную в правом верхнем углу bottom sheet, будет красивее (вот пример http://stackoverflow.com/questions/24459352/how-can-i-add-the-new-floating-action-button-between-two-widgets-layouts )

Re: idec mobile

btimofeev — 2017-04-15 17:50:33

btimofeev -> vit01

vit01> Ещё, кстати, поправил пиктограмму для уведомлений

Ты её кажется не закоммитил, программа теперь из-за этого не собирается.

Re: idec mobile

btimofeev — 2017-03-21 10:15:22

btimofeev -> Andrew Lobanov

А если в настройках ноды выставить галку на "качать индекс дальше, если все сообщения новые", не помогает?

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

Re: idec mobile

btimofeev — 2017-03-21 11:29:42

btimofeev -> Andrew Lobanov

AL> Есть ли возможность экспорта/импорта конфига?

Открой боковое меню -> дополнительно -> вкладка база данных -> в самом низу экспорт/импорт конфига. Вообще этому функционалу место в настройках приложения, имхо.

Re: IDEC mobile

btimofeev — 2017-02-14 19:49:02

btimofeev -> vit01

Можешь добавить, что бы фетчер при отсутствии интернета сообщал об этом. Сейчас он просто пишет получено ноль сообщений.

Re: IDEC mobile

btimofeev — 2017-01-10 21:14:28

btimofeev -> vit01

А как бороться с тем что клиент качает по несколько тысяч сообщений? Я вот опять обрезал эху до 50 штук, в эту эху пришло новое сообщение и клиент скачал после этого ~1500.

Re: IDEC mobile

btimofeev — 2017-01-09 22:19:11

btimofeev -> vit01

У меня какой-то странный баг случился. В один прекрасный момент клиент просто перестал скачивать сообщения. В логе писал одну строку fetch ..., затем одну строку removed pipe.2032 и потом сразу получено сообщений 0. Хотя при этом приходили уведомления, что есть новые сообщения. Так продолжалось пару дней.

Я сделал экспорт настроек, удалил данные и кэш приложения, затем импортировал настройки. Теперь если в эхе нет новых сообщений, то приложение не скачивает ничего, если в эхе появляется новое сообщение приложение скачивает сразу по 2-3 тысячи последних сообщений.

Re: IDEC mobile

btimofeev — 2016-12-19 18:59:51

btimofeev -> vit01

Кстати, по-моему, еще в notification'е тоже неправильно считает. У меня показывает большее количество сообщений, чем скачивает. Возможно просто складывается количество одинаковых сообщений из разных нод. Надо будет тоже поправить.

Re: Android клиент

btimofeev — 2016-11-06 22:28:55

btimofeev -> vit01

Писал тебе в jabber, но возможно сообщение не дошло, повторю здесь:

У меня после открытия навдравера свайпом (как в телеграме) сразу же открывается активити правки эх. Видимо срабатывает еще и долгое нажатие. Это происходит не каждый раз, но очень часто.

Re: idec-mobile

btimofeev — 2016-11-07 16:03:20

btimofeev -> vit01

vit01> Если ты прерываешь/убиваешь приложение во время чтения содержимого эхи, то метка не ставится. Вызов транзакции в БД с "прочтением" осуществляется только при нажатии кнопки "назад".

Предлагаю изменить это поведение. У меня постоянно куча непрочитанных остается в эхах с RSS после того как я просмотрел сообщения. Очень неудобно.

Re: Android клиент

btimofeev — 2016-11-03 18:55:13

btimofeev -> vit01

vit01> Но если уж и их хочется забэкапить, то проще будет вытащить sqlite-базу из /data/data/vit01.idecmobile/ и переносить именно её.

Это, кажется, будет работать только у кого есть root.

Re: Android клиент

btimofeev — 2016-11-04 16:08:27

btimofeev -> vit01

vit01> * Уведомления теперь приходят единые для всех станций (дабы больше не плодить огромное их количество)

Еще хорошо, что бы уведомления не создавались заново, а обновляли предыдущее уведомление, если оно еще есть на экране.

Re: Android клиент

btimofeev — 2016-10-23 21:52:50

btimofeev -> vit01

Протестирую, но не сегодня уже. И что там может быть не так как на шестёрке?

Кстати, я тут включил уведомления для сообщений, а у меня почему-то сразу 2 уведомления выскакивает. Возможно это из-за двух нод. Можно как-то объединить в одно?

Re: Android клиент

btimofeev — 2016-10-25 19:20:04

btimofeev -> vit01

Еще один баг пару раз мне встречался: при переключении ноды список эх остается от предыдущей ноды и сколько не пробуй выбрать любую другую ноду список эх не меняется. При этом в навдравере показывает, что нода выбрана другая. Как баг воспроизвести не знаю.

Re: Android клиент

btimofeev — 2016-10-25 15:30:44

btimofeev -> vit01

Подскажи, зачем нужна вот эта строчка https://github.com/vit1-irk/idec-mobile/blob/master/app/src/main/java/vit01/idecmobile/MainActivity.java#L70

Ты не пробовал сделать изменение темы "на лету", без перезапуска приложения?

Re: Android клиент

btimofeev — 2016-10-23 08:48:10

btimofeev -> vit01

vit01> Кстати, у тебя есть девайс или эмулятор на конкретно Android 5.0 или 5.1?

На девайсе у меня 6.0.1, а на эмулятор могу любую версию поставить, это дело пяти минут.

Re: Android клиент

btimofeev — 2016-10-22 22:55:04

btimofeev -> vit01

Еще по поводу тем:

1. В дефолтной теме в активити настройки станций в экшенбаре название станции черного цвета на фиолетовом фоне не очень выглядит.

2. В полностью белой теме надпись и иконки в экшенбаре черные, а бургер, стрелка и три точки серые. Надо, наверное, всё серым сделать.

Re: Android клиент

btimofeev — 2016-10-22 22:27:13

btimofeev -> vit01

vit01> * Добавлена тёмная тема

Очень симпатичная. Перейду на неё. Только в навдравере на мой взгляд не очень сочетаются с остальной темой выделение, разделительная черта внизу и фон картинки справа вверху - может быть они бы лучше смотрелись с цветом экшенбара, но это нужно проверить.