Added Y flip option to the coordinates

src/tiled/abstractobjecttool.cpp

@@ -37,6 +37,7 @@
 #include "tile.h"
 #include "tmxmapformat.h"
 #include "utils.h"
+#include "invertyaxishelper.h"
 
 #include <QFileDialog>
 #include <QKeyEvent>
@@ -160,7 +161,10 @@ void AbstractObjectTool::mouseMoved(const QPointF &pos,
     const QPointF tilePosF = mapDocument()->renderer()->screenToTileCoords(offsetPos);
     const int x = qFloor(tilePosF.x());
     const int y = qFloor(tilePosF.y());
-    setStatusInfo(QString(QLatin1String("%1, %2 (%3, %4)")).arg(x).arg(y).arg(pixelPos.x()).arg(pixelPos.y()));
+    setStatusInfo(QString(QLatin1String("%1, %2 (%3, %4)")).arg(x)
+                                                           .arg(InvertYAxisHelper().tileY(y))
+                                                           .arg(pixelPos.x())
+                                                           .arg(InvertYAxisHelper(MapDocumentActionHandler::instance()->mapDocument()->map()).pixelY(pixelPos.y())));
 }
 
 void AbstractObjectTool::mousePressed(QGraphicsSceneMouseEvent *event)

src/tiled/abstracttiletool.cpp

@@ -28,6 +28,7 @@
 #include "tile.h"
 #include "tilelayer.h"
 #include "tilestamp.h"
+#include "invertyaxishelper.h"
 
 #include <QtMath>
 
@@ -146,7 +147,7 @@ void AbstractTileTool::updateStatusInfo()
 
         setStatusInfo(QString(QLatin1String("%1, %2 [%3]"))
                       .arg(mTilePosition.x())
-                      .arg(mTilePosition.y())
+                      .arg(InvertYAxisHelper().tileY(mTilePosition.y()))
                       .arg(tileIdString));
     } else {
         setStatusInfo(QString());

src/tiled/invertyaxishelper.h

@@ -0,0 +1,64 @@
+/*
+ * invertyaxishelper.h
+ * Copyright 2013, Thorbjørn Lindeijer <[email protected]>
+ * Copyright 2018, Adrian Frances <[email protected]>
+ *
+ * This file is part of Tiled.
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License as published by the Free
+ * Software Foundation; either version 2 of the License, or (at your option)
+ * any later version.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+ * more details.
+ *
+ * You should have received a copy of the GNU General Public License along with
+ * this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#pragma once
+
+#include "preferences.h"
+#include "mapdocumentactionhandler.h"
+
+namespace Tiled{
+    namespace Internal {
+
+        class InvertYAxisHelper
+        {
+        public:
+            // Constructors
+            InvertYAxisHelper()  {}
+            InvertYAxisHelper(Map* m) : target(m) {}
+
+            // Inverts Y coordinate in grid
+            qreal tileY(qreal y) const
+            {
+                // Check if Invert Y Axis is set
+                if(Preferences::instance()->invertYAxis())
+                {
+                    return MapDocumentActionHandler::instance()->mapDocument()->map()->height() - 1 - y;
+                }
+                return y;
+            }
+
+            // Inverts Y coordinate in pixels
+            qreal pixelY(qreal y) const
+            {
+                // Obtain the map document
+                if(Preferences::instance()->invertYAxis())
+                {
+                    return ((target->height() + 1) * target->tileHeight()) - y;
+                }
+                return y;
+            }
+
+        private:
+            Map* target;
+        };
+
+    } // Namespace Internal
+} // Namespace Tiled

src/tiled/mapobjectmodel.cpp

@@ -29,6 +29,7 @@
 #include "mapdocument.h"
 #include "objectgroup.h"
 #include "renamelayer.h"
+#include "invertyaxishelper.h"
 
 #include <QApplication>
 #include <QPalette>
@@ -129,7 +130,7 @@ QVariant MapObjectModel::data(const QModelIndex &index, int role) const
                 return QLatin1Char('(')
                         + QString::number(mapObject->x())
                         + QLatin1String(", ")
-                        + QString::number(mapObject->y())
+                        + QString::number(InvertYAxisHelper().tileY(mapObject->y()))
                         + QLatin1Char(')');
             }
             break;

src/tiled/preferences.cpp

@@ -93,6 +93,7 @@ Preferences::Preferences()
     mLanguage = stringValue("Language");
     mUseOpenGL = boolValue("OpenGL");
     mWheelZoomsByDefault = boolValue("WheelZoomsByDefault");
+    mInvertYAxis = boolValue("InvertYAxis");
     mObjectLabelVisibility = static_cast<ObjectLabelVisiblity>
             (intValue("ObjectLabelVisibility", AllObjectLabels));
     mLabelForHoveredObject = boolValue("LabelForHoveredObject", false);
@@ -683,6 +684,17 @@ void Preferences::setWheelZoomsByDefault(bool mode)
     mSettings->setValue(QLatin1String("Interface/WheelZoomsByDefault"), mode);
 }
 
+void Preferences::setInvertYAxis(bool enabled)
+{
+    if(mInvertYAxis == enabled)
+        return;
+
+    mInvertYAxis = enabled;
+    mSettings->setValue(QLatin1String("Interface/InvertYAxis"), enabled);
+
+    emit invertYAxisChanged();
+}
+
 bool Preferences::boolValue(const char *key, bool defaultValue) const
 {
     return mSettings->value(QLatin1String(key), defaultValue).toBool();

src/tiled/preferences.h

@@ -167,6 +167,8 @@ class Preferences : public QObject
 
     bool wheelZoomsByDefault() const;
 
+    bool invertYAxis() const;
+
     /**
      * Provides access to the QSettings instance to allow storing/retrieving
      * arbitrary values. The naming style for groups and keys is CamelCase.
@@ -190,6 +192,7 @@ public slots:
     void setOpenLastFilesOnStartup(bool load);
     void setPluginEnabled(const QString &fileName, bool enabled);
     void setWheelZoomsByDefault(bool mode);
+    void setInvertYAxis(bool enabled);
 
     void clearRecentFiles();
 
@@ -229,6 +232,8 @@ public slots:
 
     void checkForUpdatesChanged();
 
+    void invertYAxisChanged();
+
 private:
     Preferences();
     ~Preferences() override;
@@ -282,6 +287,7 @@ public slots:
     bool mIsPatron;
     bool mCheckForUpdates;
     bool mWheelZoomsByDefault;
+    bool mInvertYAxis;
 
     static Preferences *mInstance;
 };
@@ -392,6 +398,11 @@ inline bool Preferences::wheelZoomsByDefault() const
     return mWheelZoomsByDefault;
 }
 
+inline bool Preferences::invertYAxis() const
+{
+    return mInvertYAxis;
+}
+
 } // namespace Internal
 } // namespace Tiled
 

src/tiled/preferencesdialog.cpp

@@ -111,6 +111,8 @@ PreferencesDialog::PreferencesDialog(QWidget *parent)
             preferences, &Preferences::setUseOpenGL);
     connect(mUi->wheelZoomsByDefault, &QCheckBox::toggled,
             preferences, &Preferences::setWheelZoomsByDefault);
+    connect(mUi->invertYAxis, &QCheckBox::toggled,
+            preferences, &Preferences::setInvertYAxis);
 
     connect(mUi->styleCombo, static_cast<void(QComboBox::*)(int)>(&QComboBox::currentIndexChanged),
             this, &PreferencesDialog::styleComboChanged);
@@ -170,6 +172,7 @@ void PreferencesDialog::fromPreferences()
     if (mUi->openGL->isEnabled())
         mUi->openGL->setChecked(prefs->useOpenGL());
     mUi->wheelZoomsByDefault->setChecked(prefs->wheelZoomsByDefault());
+    mUi->invertYAxis->setChecked(prefs->invertYAxis());
 
     // Not found (-1) ends up at index 0, system default
     int languageIndex = mUi->languageCombo->findData(prefs->language());

src/tiled/preferencesdialog.ui

@@ -225,6 +225,13 @@
             </property>
            </widget>
           </item>
+          <item row="7" column="0">
+            <widget class="QCheckBox" name="invertYAxis">
+             <property name="text">
+                <string>Invert Y axis coordinates</string>
+             </property>
+            </widget>
+          </item>
          </layout>
         </widget>
        </item>
@@ -441,6 +448,7 @@
   <tabstop>objectLineWidth</tabstop>
   <tabstop>openGL</tabstop>
   <tabstop>wheelZoomsByDefault</tabstop>
+  <tabstop>invertYAxis</tabstop>
   <tabstop>styleCombo</tabstop>
   <tabstop>baseColor</tabstop>
   <tabstop>selectionColor</tabstop>

src/tiled/propertybrowser.cpp

@@ -62,6 +62,7 @@
 #include "variantpropertymanager.h"
 #include "wangcolormodel.h"
 #include "wangset.h"
+#include "invertyaxishelper.h"
 
 #include <QtGroupPropertyManager>
 
@@ -102,6 +103,9 @@ PropertyBrowser::PropertyBrowser(QWidget *parent)
 
     connect(Preferences::instance(), &Preferences::objectTypesChanged,
             this, &PropertyBrowser::objectTypesChanged);
+
+    connect(Preferences::instance(), &Preferences::invertYAxisChanged,
+            this, &PropertyBrowser::invertYAxisChanged);
 }
 
 void PropertyBrowser::setObject(Object *object)
@@ -323,6 +327,12 @@ void PropertyBrowser::wangSetChanged(Tileset *tileset, int index)
         updateProperties();
 }
 
+void PropertyBrowser::invertYAxisChanged()
+{
+    if (mObject && mObject->typeId() == Object::MapObjectType)
+        updateProperties();
+}
+
 static QVariant predefinedPropertyValue(Object *object, const QString &name)
 {
     QString objectType;
@@ -1020,7 +1030,7 @@ QUndoCommand *PropertyBrowser::applyMapObjectValueTo(PropertyId id, const QVaria
     }
     case YProperty: {
         const QPointF oldPos = mapObject->position();
-        const QPointF newPos(oldPos.x(), val.toReal());
+        const QPointF newPos(oldPos.x(), InvertYAxisHelper(mMapDocument->map()).pixelY(val.toReal()));
         command = new MoveMapObject(mMapDocument, mapObject, newPos, oldPos);
         break;
     }
@@ -1608,7 +1618,7 @@ void PropertyBrowser::updateProperties()
         if (auto visibleProperty = mIdToProperty[VisibleProperty])
             visibleProperty->setValue(mapObject->isVisible());
         mIdToProperty[XProperty]->setValue(mapObject->x());
-        mIdToProperty[YProperty]->setValue(mapObject->y());
+        mIdToProperty[YProperty]->setValue(InvertYAxisHelper(MapDocumentActionHandler::instance()->mapDocument()->map()).pixelY(mapObject->y()));
 
         if (flags & ObjectHasDimensions) {
             mIdToProperty[WidthProperty]->setValue(mapObject->width());

src/tiled/propertybrowser.h

@@ -101,6 +101,7 @@ private slots:
     void tileTypeChanged(Tile *tile);
     void terrainChanged(Tileset *tileset, int index);
     void wangSetChanged(Tileset *tileset, int index);
+    void invertYAxisChanged();
 
     void propertyAdded(Object *object, const QString &name);
     void propertyRemoved(Object *object, const QString &name);