add camera screen styles + color filters by ve-sdk-android · Pull Request #2 · ve-sdk-android/ve-sdk-android-integration-sample

ve-sdk-android · 2020-10-22T15:40:42Z

No description provided.

GlebPBanuba

А что за ресурсы C1.png ? это иконки?

ve-sdk-android · 2020-10-23T07:37:41Z

А что за ресурсы C1.png ? это иконки?

В папке bnb-resources/luts -> квадратные исходники для лутов, в папке drawables -> соответствующие им иконки для редактора

GlebPBanuba · 2020-10-22T20:50:28Z

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="utf-8"?>
+<shape xmlns:android="http://schemas.android.com/apk/res/android"


а зачем мы это здесь переопределяем?

Т.к. переопределяли полностью лейаут алерта, нам нужно переопределить и бэкграунд для кнопок

GlebPBanuba · 2020-10-22T20:51:08Z

@@ -0,0 +1,15 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"


а эти иконки дала Лена?

да, это из ее архива, svg конвертированные в vector через андроид студию

GlebPBanuba · 2020-10-22T20:51:27Z

@@ -0,0 +1,44 @@
+<?xml version="1.0" encoding="utf-8"?>
+<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"


А зачем?

В ve-ui модуле лейаут не совсем такой, каким мы его хотим видеть и поэтому переопределяем его. В Демо точно такой же подход. Судя по всему мы реализовывали его по требованиям Яндекса, но не устанавливали его как вариант по умолчанию.

Нужно заталкать в SDK чтобы не переопределять каждый раз, но мы должны предоставить возможность как переопределять.

GlebPBanuba

Давай оставим луты - черно белый и какой нибудь цветной.

GlebPBanuba · 2020-10-23T15:56:10Z

+import com.banuba.sdk.cameraui.data.CameraRecordingAnimationProvider
+import com.banuba.sdk.core.ui.ext.dimenPx
+
+class CustomRecordingAnimationProvider (context: Context) : CameraRecordingAnimationProvider {


Нужно подумать как избавиться от этого в интеграции

GlebPBanuba · 2020-10-23T15:56:47Z

@@ -0,0 +1,124 @@
+package com.banuba.example.integrationapp.videoeditor.custom.widget
+
+import android.animation.ValueAnimator


Нужно постараться исключить из интеграции

GlebPBanuba · 2020-10-23T15:57:11Z

@@ -0,0 +1,196 @@
+package com.banuba.example.integrationapp.videoeditor.custom.widget
+
+import android.animation.AnimatorSet


Нужно исключить из интеграции

GlebPBanuba · 2020-10-23T15:57:49Z

@@ -0,0 +1,19 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"


Хотелось бы исключить из интеграции

GlebPBanuba · 2020-10-23T15:58:29Z

@@ -0,0 +1,26 @@
+<?xml version="1.0" encoding="utf-8"?>


Можно ли это исключить из интеграции?

GlebPBanuba · 2020-10-23T15:58:46Z

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="utf-8"?>


Можно ли исключить из интеграции?

GlebPBanuba · 2020-10-23T15:59:02Z

@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="utf-8"?>


Можно ли исключить из интеграции?

GlebPBanuba reviewed Oct 22, 2020

View reviewed changes

ve-sdk-android force-pushed the integration/camera_screen branch from b8ac545 to 8a60f3b Compare October 23, 2020 09:36

GlebPBanuba reviewed Oct 23, 2020

View reviewed changes

ve-sdk-android force-pushed the integration/camera_screen branch from 8a60f3b to 6d9ce5a Compare October 23, 2020 10:53

GlebPBanuba reviewed Oct 23, 2020

View reviewed changes

add camera screen styles + color filters

9e29a49

ve-sdk-android force-pushed the integration/camera_screen branch from 6d9ce5a to 9e29a49 Compare October 23, 2020 16:08

		@@ -0,0 +1,6 @@
		<?xml version="1.0" encoding="utf-8"?>
		<shape xmlns:android="http://schemas.android.com/apk/res/android"

		@@ -0,0 +1,15 @@
		<vector xmlns:android="http://schemas.android.com/apk/res/android"

		@@ -0,0 +1,44 @@
		<?xml version="1.0" encoding="utf-8"?>
		<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"

		@@ -0,0 +1,124 @@
		package com.banuba.example.integrationapp.videoeditor.custom.widget

		import android.animation.ValueAnimator

		@@ -0,0 +1,196 @@
		package com.banuba.example.integrationapp.videoeditor.custom.widget

		import android.animation.AnimatorSet

		@@ -0,0 +1,19 @@
		<vector xmlns:android="http://schemas.android.com/apk/res/android"

Conversation

ve-sdk-android commented Oct 22, 2020

Uh oh!

GlebPBanuba left a comment

Choose a reason for hiding this comment

Uh oh!

ve-sdk-android commented Oct 23, 2020

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ve-sdk-android Oct 23, 2020 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

GlebPBanuba left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

ve-sdk-android Oct 23, 2020 •

edited

Loading