Jump to content

Carousel wasted space? ...


Recommended Posts

Posted

Never had a time to report this, and quite frankly I have no idea if this is something that should be reported upstream or it's just a config thing. (was too lazy to grab 'vanilla' xvm config to test it)

 

Carousel has a bit of extra space on the left and right, is there something that could be done for this ? Or it's on purpose in order to properly size the available viewport for full number of cells ? (In my case i can see 10 tanks at the same time (since i'm using 2 rows))

 

I'm talking about space left from filters and right of 'slider'.

 

2015-10-29_16-58-29.png?dl=0

  • Moderator
Posted (edited)

Probably just filler space, so the tank icons fit properly.

 

You could always give it a try, by changing the two numbers in the padding section:

\res_mods\configs\xvm\Aslain\carousel.xc

(Recommended using Notepad++)

 

Edit: The padding is around the tank icons on the carousel.

Edited by Quaksen
Posted (edited)

Probably just filler space, so the tank icons fit properly.

 

Yeah, It's just a space that couldn't be used for even number of cells.

I guess this is related code for it https://bitbucket.org/XVM/xvm/src/57519fdcb09458350fb76ca23f95776f2ebb88d5/src/xvm/src/xvm/tcarousel_ui/UI_TankCarousel.as?at=default&fileviewer=file-view-default#UI_TankCarousel.as-497

 

edit:

In my case I've managed to get another two slots visible by just setting horizontal padding to 2 instead of 10.

diff --git a/carousel.xc b/carousel.xc
index cc3363b..863f667 100644
--- a/carousel.xc
+++ b/carousel.xc
@@ -8,7 +8,7 @@
     "rows": 2,
     // Spacing between carousel cells.
     "padding": {
-        "horizontal": 10,
+        "horizontal": 2,
         "vertical": 2
     },
     // true - show filters only if all tanks do not fit on the screen.
Edited by thevbm

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use and Privacy Policy.