Add an option to change chart tick mark type

[RokasDie]

Hello,

One of the feature which I saw was missing is to change the minor and major tick mark position for charts. For example: whether the tick marks should be inside, outside or cross axis.

Sadly, I am fairly new to development and Github in general, so I do not know how to update the documentation. Readme.md seemed only to have links to the "Adding Charts" page. So any tips or help on how to do that would be really helpful.

Regards,
Rokas

[gitbrent]

Hi @RokasDie

Thanks for the contribution. The docs are in a different branch, so no worries there.

Can you provide demo code and output for the new types you've added so I can validate that they function correctly?

Check examples/pptxgenjs-demo.js for loads of chart demo code you can use as a basis for a couple of examples.

[RokasDie]

Hello, @gitbrent

Please see some results of my testing with the changed branch:
tickMarksCross.pptx
tickMarksOutside.pptx
tickMarksInside.pptx
tickMarksNone.pptx

I also attach the code which was used:

var PptxGenJS = require("pptxgenjs");

var pptx = new PptxGenJS();
var slide = pptx.addNewSlide('MASTER_SLIDE');

var arrDataRegions = [
    {
        name  : 'Region 1',
        labels: ['May', 'June', 'July', 'August'],
        values: [26, 53, 100, 75]
    },
    {
        name  : 'Region 2',
        labels: ['May', 'June', 'July', 'August'],
        values: [43.5, 70.3, 90.1, 80.05]
    }
];

// TOP-LEFT: H/bar
var optsChartBar1 = { x:0.5, y:0.6, w:6.0, h:3.0,
    barDir: 'bar',
    border: { pt:'3', color:'00EE00' },
    fill: 'F1F1F1',

    catAxisLabelColor   : 'CC0000',
    catAxisLabelFontFace: 'Helvetica Neue',
    catAxisLabelFontSize: 14,
    catAxisOrientation  : 'maxMin',

    titleColor   : '33CF22',
    titleFontFace: 'Helvetica Neue',
    titleFontSize: 24,
    // commands: "in", "out", "cross", "none"
    valAxisMinorUnitType: "none",
    valAxisMajorUnitType: "none"
};
slide.addChart( pptx.charts.BAR, arrDataRegions, optsChartBar1 );

pptx.save('./PowerPoints/testing.pptx');

Please tell me if something else needs to be checked.

Regards,
Rokas

[gitbrent]

Thanks @RokasDie !

This has been added to the 3.0 codebase.