Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

I1228 Add y-scroll-bar to webpage #1230

Merged

Conversation

tst-eclamar
Copy link
Collaborator

@tst-eclamar tst-eclamar commented Mar 19, 2019

Fixes #1228

Insufficient screen real estate for button on glc-assembly webpage under Firefox:

  1. Navigate to Fusion Server admin page (127.0.0.1/admin)
  2. Click on tool bar button ("gear" icon on right-hand side of page)
  3. Select "Glc Assembly" - a pop-up window appears titled "GLC assembly tool"
  4. Fill in 'name', 'title', and dummy text into 'polygon' fields.
  5. Click on 'Layers' "3d" radio button
  6. Scroll to bottom of page
  7. View "Assemble" button at bottom of window.

@googlebot googlebot added the cla: yes Manual verification that all contributors have signed the CLA. label Mar 19, 2019
@tst-nfarah-zz tst-nfarah-zz added this to the 5.3.0 milestone Mar 20, 2019
@tst-nfarah-zz
Copy link
Collaborator

Is there a Github ticket for this issue being fixed ? If yes please reference it here, otherwise we need to create it first.

@tst-nfarah-zz
Copy link
Collaborator

Document the fix in release notes

Copy link
Collaborator

@tst-nfarah-zz tst-nfarah-zz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Document in release notes an reference Github ticket being fixed

@tst-tcalder
Copy link
Collaborator

Nice Work. Scroll bar is there and works like it should.

@@ -152,6 +152,14 @@ <h5>
Added an experimental option that fixes several memory leaks related to our usage of the Xerces library.
</td>
</tr>
<tr>
<td>1228</td>
<td>Insufficient screen real estate for 'Assemble' button</td>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's re-word this to some thing like "Assemble button is not fully visible making difficult to use the GLC assembly tool"

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i don't see the change. Was it pushed ?

@tst-ccamp
Copy link
Collaborator

I see two approvals, so merging.

@tst-ccamp tst-ccamp merged commit 7f17ffb into google:master Mar 26, 2019
husf-dsheremata added a commit to husf-dsheremata/earthenterprise that referenced this pull request Jun 4, 2019
* 'master' of https://github.com/google/earthenterprise:
  1270 Fix xml parsing in common.sh (google#1271)
  1192 update openjpeg to 2.3.1 (google#1269)
  Documented required boost dependency for Ubuntu 14 (google#1264)
  [WIP] Resolve issue google#798 - investigate Qt build performance (google#1245)
  Copy uninstall scripts (google#1232)
  refresh commit
  Issue google#1255 - Limits internal version string to 3 parts to fix various errors with patch releases (google#1256)
  Move build number to release rpm (google#1252)
  fix version
  link up 5.3.1 doc folder (google#1249)
  I1246 create 5.3.1 doc folder from copy of 5.3.0 doc folder (google#1248)
  change version files over to 5.3.1 (google#1247)
  start of 5.3.1
  Fixes google#279 - Admin Console password will be preserved on GEE server upgrade (google#1243)
  google#1201 - Fix issue with setting source date when using .kip directories (google#1234)
  I1228 Add y-scroll-bar to GLC Assembly webpage (google#1230)
  google#1236 Polygons constructor can accept KML as a string (google#1239)
  I1225 Re-add GROUPNAME to install_server.sh (google#1233)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Manual verification that all contributors have signed the CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants