Skip to content

Update Magic-HD-Night/Noire for new VirtualKeyBoard#54

Merged
AbuBaniaz merged 3 commits intoOpenViX:masterfrom
IanSav:patch-1
Apr 14, 2019
Merged

Update Magic-HD-Night/Noire for new VirtualKeyBoard#54
AbuBaniaz merged 3 commits intoOpenViX:masterfrom
IanSav:patch-1

Conversation

@IanSav
Copy link
Copy Markdown
Contributor

@IanSav IanSav commented Apr 14, 2019

  • [Magic-HD-Night] Update VirtualKeyBoard images

    Add new images for the updated VirtualKeyBoard.

  • [Magic-HD-Noire] Update VirtualKeyBoard images

    Add new images for the updated VirtualKeyBoard.

  • [CommonSkin.xml] VirtualKeyBoard update

    Update skin for compatibility with the new VirtualKeyBoard.

IanSav and others added 3 commits April 14, 2019 17:06
Update skin for compatibility with the new VirtualKeyBoard.
Add new images for the updated VirtualKeyBoard.
Add new images for the updated VirtualKeyBoard.
@AbuBaniaz
Copy link
Copy Markdown
Contributor

The png files in the defaultfallback skin are used. Do you still want this merged?

@IanSav
Copy link
Copy Markdown
Contributor Author

IanSav commented Apr 14, 2019

The changes to the skin XML file are needed.

When I tested the new XML file the images did not get loaded. When I added the images they were found.

Can you test just the XML change and see if you experience similar results.

@AbuBaniaz
Copy link
Copy Markdown
Contributor

I have tried this several times, it works without any changes.

I am presuming that the person on forum removed the fallback skin. If I rename the default skin folder, keyboard does not show.

Not a problem merging, but please be sure.

@IanSav
Copy link
Copy Markdown
Contributor Author

IanSav commented Apr 14, 2019

Please give the merge a try. We can always revert if there is a problem.

Given that this skin has been abandoned by the person you copied it from Rob perhaps it should be removed entirely and ask the user to move onto one of Rob's supported skins?

@AbuBaniaz
Copy link
Copy Markdown
Contributor

You can't even uninstall the default skin. So was a manual removal.

It works as it is. I'll someone else decide.

IMO, we are better off making other skins same as this instead of duplication. Pity there is no 1080 default skin

@Huevos
Copy link
Copy Markdown
Contributor

Huevos commented Apr 14, 2019

Sooner or later we need to get round to using SVG images.

@IanSav
Copy link
Copy Markdown
Contributor Author

IanSav commented Apr 14, 2019

I tried to propose a multi resolution default skin. No-one seemed interested.

@IanSav
Copy link
Copy Markdown
Contributor Author

IanSav commented Apr 14, 2019

By the way, I have the new VirtualKeyBoard images in skin_default yet the Magic-HD-Night skin could not load the images. The XML file is definitely in need of change.

@IanSav
Copy link
Copy Markdown
Contributor Author

IanSav commented Apr 14, 2019

@AbuBaniaz: Did you test the "ViX-HD-Night" skin or the "Magic-HD-Night" skin? The problems are with "Magic-HD-Night" (and "Magic-HD-Noire").

@AbuBaniaz
Copy link
Copy Markdown
Contributor

Ian, there is no problem. I checked Delagroov's skin.

@AbuBaniaz AbuBaniaz merged commit 12c8983 into OpenViX:master Apr 14, 2019
@AbuBaniaz
Copy link
Copy Markdown
Contributor

Weird, two receivers were fine, third was not. So merged it.

@IanSav IanSav deleted the patch-1 branch April 14, 2019 23:36
@IanSav
Copy link
Copy Markdown
Contributor Author

IanSav commented Apr 15, 2019

The changes I submitted should be safe and fine even if the skin_default images are working. :)

Huevos added a commit that referenced this pull request Apr 15, 2019
This reverts commit 12c8983.

Reverted as it breaks the build.
configure.ac must be edited when adding folders to a Makefile. Please resubmit pull request when corrected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants