[needs-docs] Use correct open form icon in identify results toolbar

Before we used a different (non obvious) icon for "open form"
in the toolbar. This changes the toolbar icon to reuse the
same (clearer) icon which is used when expanding out the
tree and finding the "show feature form" action.

I've also moved the Open Form action to be first in the
toolbar, since it's the most frequently used action.

Author: nyalldawson

src/ui/qgsidentifyresultsbase.ui

@@ -62,10 +62,10 @@
          <property name="floatable">
           <bool>false</bool>
          </property>
+         <addaction name="mOpenFormAction"/>
          <addaction name="mExpandAction"/>
          <addaction name="mCollapseAction"/>
          <addaction name="mExpandNewAction"/>
-         <addaction name="mOpenFormAction"/>
          <addaction name="mClearResultsAction"/>
          <addaction name="mActionCopy"/>
          <addaction name="mActionPrint"/>
@@ -291,7 +291,7 @@
   <action name="mOpenFormAction">
    <property name="icon">
     <iconset resource="../../images/images.qrc">
-     <normaloff>:/images/themes/default/mActionPropertyItem.svg</normaloff>:/images/themes/default/mActionPropertyItem.svg</iconset>
+     <normaloff>:/images/themes/default/mActionFormView.svg</normaloff>:/images/themes/default/mActionFormView.svg</iconset>
    </property>
    <property name="text">
     <string>Open Form</string>