Skip to content

Does not render properly in 2.3.2 #74

Open

Description

Revo 2.3.2, Image+ 2.2.0 beta.
rendering_232

Activity

  1. exside commented on Dec 29, 2014

    @exside

    jap, it's a mess and my bad in part (like in re-doing the inputs for 2.3)... wanted to fix this myself in spare time but currently I dont have such thing^^

  2. alanpich commented on Dec 31, 2014

    @alanpich
    Owner

    wow... yeah i can see that being a bit of an issue for usability....
    sorry i've been absent on this project for a (long) while; I've been elsewhere and distracted etc etc.

    From the screenshot it looks like a clash between the css hacks that were put in place (by me) to sort layout issues in the 2.2* releases against the nice new structured styles of the 2.3 releases. I doubt it's ExtJs causing that top-margin issue - take a look at the image+ stylesheets.

    The clipping of the icon on the right hand side of the screenshot is definitely caused by the ExtJs framework - it uses explicit container widths for all panels and clips anything that overflows.

    Has anyone got a fork of this fix on the go for 2.3 or do i need to brush the dust off my Modx?

  3. yuka123 commented on Jan 1, 2015

    @yuka123

    sottwell you don't have to do that in the "doodles" tutorial. It's far more simpler. You fork this repo to your account. Then there is a _build as you can see in the repo. Just set it up in your webserver directory and run it. It will create the latest package. Now this puts everything in place already for you to work on the issues.(ofc you have to install the package) Once you create any changes on your local installation of the build you can just push those to your repo and propous the pull request to the original repo.

    Best, yuka

  4. Jako commented on Mar 25, 2015

    @Jako

    Have done some compatibility work on the 2.2.0 version from MODX Extras and commited it to master on https://github.com/Jako/ImagePlus.

    The folder renaming was somehow necessary to use GitPackageManagement for build. At that time I changed tvimageplus to imageplus in the whole package.

    The main changes here:

    Maybe they are also valid for 3.0.x branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions