Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes direction to be render-forward for all plots and themes, fixes #18... #109

Closed
wants to merge 109 commits into from

Conversation

Kagetsume
Copy link
Contributor

This is related to enhancement/defect: https://bugs.dojotoolkit.org/ticket/18061

It is a simple patch to the plots to reverse the draw order so it draws theme-forward consistently (theme entry 0 goes to series 0, theme entry 1 to series 1, and so on). Prior to this some plots rendered reverse order, some rendered forward. It made it inconsistent when displaying several cart types together in terms of their draw color sequence. This also makes it simpler to build new themes as the draw order makes more sense.

I verified in the current/modern dojo code base this does not affect the way stack charts render, they still render properly stacked, just with theme color draws in forward order.

Kagetsume and others added 30 commits May 29, 2014 11:07
Line endings in dojo (and most repositories) must be \n, not \n\r.
Necessary to generate API doc.
Tweak was overwritten by ab5ea21 (refs #17822).
Chrome clears the file input value when the file selection dialog is cleared.
If a file was already chosen this creates a change, but the data array
provided to `onChange` wasn't accounting for this so a dummy file object
gets passed instead of an array.
…tion/deserialization of a VML surface.

Squashed commit of the following:

commit 7c87313
Author: Jared Jurkiewicz <jared.jurkiewicz@gmail.com>
Date:   Mon Jul 28 11:09:12 2014 -0400

    Fix patth repeated duplication error in VML renderer, fixes #16830
commit d60e612
Author: Jared Jurkiewicz <jared.jurkiewicz@gmail.com>
Date:   Tue Jun 10 11:21:50 2014 -0400

    Fix issue with dojox/html/metrics and font size, fixes #18080
commit c7e6a45
Author: Nicola Rizzo <nicola.rizzo@gmail.com>
Date:   Sat Jul 26 19:51:30 2014 +0200

    corrects the year number, fixes #17835; test updated
commit 4306992
Author: Jared Jurkiewicz <jared.jurkiewicz@gmail.com>
Date:   Fri Aug 15 13:15:54 2014 -0400

    Fix issue with target node check, make sure it isn't root/body, fixes #18222
@dylans dylans added this to the 1.11 milestone Mar 25, 2015
Matt Wistrand and others added 16 commits April 19, 2015 07:34
Squashed commit of the following:

commit a475eec
Author: Arūnas Rukšnaitis <github@ruksnaitis.com>
Date:   Tue Apr 28 20:27:36 2015 +0100

    #18533 dojox/lang/functional zip.js and curry.js in AMD style (broken in 1.10.4)
Thanks to youngho (CLA on file) for patch.

Also did some lint cleanup.

Fixes #18619.
Fixes an issue where any widget using Selection.js (such as dojox/Calendar) is only able
to multi-select if you control-click, even on OSX.  (Unfortunately for them, control-click
also simulates the Windows right-click behavior, making the browser context menu appear.)
@dylans
Copy link
Member

dylans commented Jul 3, 2015

@Kagetsume do you want to go ahead and land this one?

@Kagetsume
Copy link
Contributor Author

Sure, I'll merge it today. I've been swamped. Sorry for the delay on these.

@Kagetsume Kagetsume closed this Sep 3, 2015
@Kagetsume Kagetsume reopened this Sep 3, 2015
@Kagetsume
Copy link
Contributor Author

I have merged this into master. Therefore, closing pull request!

@Kagetsume Kagetsume closed this Sep 4, 2015
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.

None yet