![]() |
Release Amber for Leia/Matrix/Nexus - Printable Version +- Kodi Community Forum (https://forum.kodi.tv) +-- Forum: Support (https://forum.kodi.tv/forumdisplay.php?fid=33) +--- Forum: Skins Support (https://forum.kodi.tv/forumdisplay.php?fid=67) +---- Forum: Amber (https://forum.kodi.tv/forumdisplay.php?fid=203) +---- Thread: Release Amber for Leia/Matrix/Nexus (/showthread.php?tid=342130) Pages:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
|
RE: Amber for Leia - bsoriano - 2019-04-25 (2019-04-25, 08:34)DarrenHill Wrote: @bsoriano - as you wish@DarrenHill , I agree with you, and I think it would be a nice addition to the skin, so I will add this as well. What do you think, an option to "Enable 2 finger right swipe for Menu" in Amber Settings? Thanks. Regards, Bart RE: Amber for Leia - Raitsa - 2019-04-25 (2019-04-24, 19:56)bsoriano Wrote:Aah, my eyes just somehow missed the option to turn the search off.(2019-04-24, 19:33)Raitsa Wrote: I like keeping things simple so I wish to not see the search option on my screen. All ok now ![]() RE: Amber for Leia - DarrenHill - 2019-04-25 @bsoriano - Something like that should work yes, as long as the python script can support both enable and disable (ie can both move the keymap xml file into the keymaps folder on enable and also remove it if disabled). But again there the file naming should be very specific to Amber to prevent any problems of overwriting or accidentally deleting something else (and the script being able to cope with files already existing or not being there if delete is requested). I must admit I'm not sure if the blade menu is accessible with just a mouse (as of course most times you have the keyboard as well to just press left arrow to get it) - I'm planning to check that tonight under Windows. Just to see if button and right-swipe are both needed, although I guess having both options will do no harm if adding the button isn't too difficult, especially if they're both switchable. If you're considering mouse-only support it might also be worth thinking about a back button as well? (two-finger left-swipe is already defined as back in the default touchscreen keymap). And maybe a home button two? Perhaps have two toggles, one to enable/disable touchscreen blade support (the keymap change) and a second to enable/disable mouse support (show/hide mouse buttons for blade/view, back and maybe home)? RE: Amber for Leia - bsoriano - 2019-04-25 (2019-04-25, 16:23)DarrenHill Wrote: @bsoriano - Something like that should work yes, as long as the python script can support both enable and disable (ie can both move the keymap xml file into the keymaps folder on enable and also remove it if disabled). But again there the file naming should be very specific to Amber to prevent any problems of overwriting or accidentally deleting something else (and the script being able to cope with files already existing or not being there if delete is requested).@DarrenHill , thanks for thinking this through in detail ![]() The script I saw does only one thing, copy the keymap from a skin folder to the userdata folder. it has no parameters or does anything else. My pythons skills are not at the level where I can add parameters to that script and functionality to remove the file from the userdata folder. I will have to study a bit and would take me a while to get it to that point. So perhaps in the time being, I can add the keymap to a skin folder and in this thread (first post and a separate one) explain to folks how they can enable this themselves. This would mean for now, no option added in Amber settings. I already created a keymap called Amber.xml that adds the gestures to the relevant windows only (videos,music,pictures,games,weather,programs,addonbrowser). I will create a "scripts" skin folder and put it there. What do you think? In terms of mouse support, I already tried (part of it is in Git already) adding an "Options" button to all of the views, and it works well, both touch (my laptop has a touch display) and mouse. I like your idea of adding also a Back and a Home button. These make sense throughout the skin, so it might take me a while to figure out where to position them that would be consistent everywhere. Any ideas? I could start by just having them in the media views. It will take me a bit to finish this, and yes, in the case of these buttons, they can definitely be switchable. Again, thanks for your ideas and I await your feedback. Regards, Bart RE: Amber for Leia - DarrenHill - 2019-04-25 @bsoriano - to be honest for the keymap I'd probably just follow the xonfluence route and silently add it at first run. My keymap version using the global level has the advantage that any other keymap which does use lower level tags (for specific windows, like your one) will override it. That said I couldn't see any definition using two-finger right swipe in the touchscreen.xml (wiki) file, which is the only one for touchscreen swipes anyway. Also if it's called Amber.xml it's towards the start of the alphabet, and keymaps are loaded in alphabetical order (with the last one overwriting any previously loaded ones). Of course for most devices that don't have a touchscreen the whole thing is irrelevant anyway ![]() Concerning buttons, I would perhaps use Estouchy or one of the other touchscreen-optimised skins (Retouched perhaps) as a guideline? There is also the Mousetuary variant of Estuary as another example. I'd probably just group them together somewhere out of the way - probably towards one of the screen corners. RE: Amber for Leia - bsoriano - 2019-04-25 (2019-04-23, 18:58)Cyberdom Wrote: Hello everyone.@Cyberdom , sorry it's taking me a bit to address the issues. I will shortly provide a fix for issue 1. For issue 4, unfortunately I do not have a PVR client that does recordings, so I cannot test this. Could you please send me a screenshot of what it looks like in Amber (that does not show the channel logo like you would like) and a screenshot of Estuary that shows the channel logo? That way I'll have an idea of what to look for in the code. Thanks. Also for issue 4, can you also please send me a screenshot of the program guide that does not display the end time? With the PVR client I am using (Simple IPTV), I do see end time for each program shown in the guide. Again, thanks. Regards, Bart RE: Amber for Leia - Cyberdom - 2019-04-25 Thank you for the fix for issue 1. For Tvheadend HTSP Client, here are the screenshots : Screenshot with the playback of a recording under Estuary with the logo of the TV channel that appears and the end time : ![]() Screenshot with Amber recording with the TV channel logo not showing : ![]() Screenshot of a live TV channel with the TV channel logo displayed. The end time is not displayed but there is just the current time : ![]() RE: Amber for Leia - bsoriano - 2019-04-26 (2019-04-25, 17:09)DarrenHill Wrote: @bsoriano - to be honest for the keymap I'd probably just follow the xonfluence route and silently add it at first run. My keymap version using the global level has the advantage that any other keymap which does use lower level tags (for specific windows, like your one) will override it. That said I couldn't see any definition using two-finger right swipe in the touchscreen.xml (wiki) file, which is the only one for touchscreen swipes anyway. Also if it's called Amber.xml it's towards the start of the alphabet, and keymaps are loaded in alphabetical order (with the last one overwriting any previously loaded ones). @DarrenHill , I went ahead and followed your suggestions (thanks!) for the mouse/touch buttons in the media views. Changes are in Git and the Amber repo. Please test and let me know what you think. You have to enable the option "Enable Touch Buttons" in Amber Settings. Thanks. @shedrock , could you please try this out as well and let me know if this works for you as well on your tablet? Thanks. Regards, Bart RE: Amber for Leia - bsoriano - 2019-04-26 (2019-04-25, 23:08)Cyberdom Wrote: Thank you for the fix for issue 1.@Cyberdom , thank you for the screenshots. I think I have fixed the issues (logo and remaining time). Please download from Git (or if you installed the Amber repo the skin should update automatically), test, and let me know if you find any additional issues. Thanks. One thing to note is that the remaining time, duration, elapsed time and finish time will not be correct or missing for live tv items that do not have EPG. Regards, Bart RE: Amber for Leia - DarrenHill - 2019-04-26 @bsoriano - tried the touch buttons. They generally seem to be working, but there are a few comments:
RE: Amber for Leia - bsoriano - 2019-04-26 (2019-04-26, 20:32)DarrenHill Wrote: @bsoriano - tried the touch buttons. They generally seem to be working, but there are a few comments:@DarrenHill , thanks for your feedback. Yes, the buttons are a little small, but I wanted to make them fit in the height of the skin's top bar. I agree the placement is not optimal either, but I'm afraid my designs skills only go so far (sorry!) and top placement was the only place on the screen that could be the same no matter what media window it was. I can certainly add the relevant buttons to the other windows. Given the current limitations of the feature, do you still think it worthwhile? Regards, Bart RE: Amber for Leia - DarrenHill - 2019-04-26 I tried with just mouse only, and there currently is no way to bring the blade menu up that way that I can see (right click does back). I'll have a look later and see if I can come up with any suitable keymapping for a mouse that would be the equivalent of the two finger swipe right on touchscreen. If there is then perhaps going that route could be an option, but if not it's a question of how many people actually exclusively use mice for controlling Kodi. I know it's not an unknown scenario, but it's a rare one. RE: Amber for Leia - bsoriano - 2019-04-26 (2019-04-26, 21:00)DarrenHill Wrote: I tried with just mouse only, and there currently is no way to bring the blade menu up that way that I can see (right click does back).@DarrenHill, the touch button I added did not work for you with the mouse? I tried it and it worked for me. Regards, Bart RE: Amber for Leia - DarrenHill - 2019-04-26 Sorry, wasn't clear. I tried using just a mouse on the standard set-up and there's no way to open the blade menu with just a mouse - keyboard or something is needed. But with the buttons enabled that worked fine in all three actions. Blade menu popped up correctly and went away correctly on cog button press. Editted to add - I've just found another mouse glitch. If you are in one of the menus (e.g. the addon browser) and right click on something, you get the context menu up. There's no way to actually back out of that with just the mouse other than to select something. At least with a normal 3-button mouse anyway (I run a 5-button Intellimouse and the left thumb-button works as back in any window there). And there's no way I think any other route is possible there, except perhaps without a specific keymap to define right click to back within the context menu window (if that's possible). RE: Amber for Leia - DarrenHill - 2019-04-26 @bsoriano - how about the Amber.xml keymap below. It makes right click in the context menu cancel/back out of the menu and a "middle click" (ie pushing the mouse wheel) open the blade menu. Plus the touch screen double-finger right swipe opening the menu from before. All specifically focussed on the navigation windows. Another option is long right-click (<longclick id="1">Menu</longclick>) but that doesn't feel at all natural. My only concern would be for people with wheel-less rodents... Edited to add - more windows need to be covered (mainly PVR ones) as they also have blade menus.
|