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

Configure Pop-up: Insert Link and Image Buttons #7

Closed
wspiking opened this issue Feb 2, 2015 · 22 comments
Closed

Configure Pop-up: Insert Link and Image Buttons #7

wspiking opened this issue Feb 2, 2015 · 22 comments
Assignees
Labels
Milestone

Comments

@wspiking
Copy link

wspiking commented Feb 2, 2015

The Insert Link and Insert Image buttons in the Configure Pop-up window don't seem to be working. Please verify.

@WashGIS
Copy link

WashGIS commented Feb 2, 2015

you beat me too it. It is not working for me either

@rscheitlin
Copy link
Collaborator

This issue will be fixed in the next release

@wspiking
Copy link
Author

wspiking commented Feb 2, 2015

Thank you Robert. One other note on the pop-up config. In the Custom pop-up config window, is there a way to add

when a user adds a hard return?

@rscheitlin
Copy link
Collaborator

Currently just type <br>. But that is something I will look at.

@WashGIS
Copy link

WashGIS commented Feb 2, 2015

Thank you

From: Robert J Scheitlin [mailto:notifications@github.com]
Sent: Monday, February 02, 2015 3:34 PM
To: cmndrbensisko/LocalLayerWidget
Cc: Rathvon, Joe
Subject: Re: [LocalLayerWidget] Configure Pop-up: Insert Link and Image Buttons (#7)

This issue will be fixed in the next release


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-72532102.

@rscheitlin rscheitlin self-assigned this Feb 3, 2015
@rscheitlin rscheitlin added the bug label Feb 3, 2015
@rscheitlin rscheitlin added this to the Version 1.3 milestone Feb 3, 2015
@wspiking
Copy link
Author

wspiking commented Feb 4, 2015

Hi Robert, I'm still having issues w/ the Insert Link and Insert Image buttons. Also, the hard return is just being added to the first line. See attached image. To make sure I reinstalled WAB Dev and loaded it up with LocalLayer 1.3 and it didn't make a difference. Let me know if you need me to do any additional testing.
Thank you sir,
Bill
capture

@rscheitlin
Copy link
Collaborator

William, Can you verify that you copied the the new version of the widget to your stemapp widgets folder and the widget folder of any app that you have already built and are attempting to use the Local Layer widget in. You can check that you actually using the new version by opening the manifest.json and check the version number in that file. If you replaced the widget in the stemapp but not in your built apps widget directory then your built app will be using the old version still. Another why right off you can tell that you are using the 1.3 vs the 1.2 is there will be a Add Tiled Layer option in the widget.

@wspiking
Copy link
Author

wspiking commented Feb 5, 2015

I'll verify this morning and get back to you. Question....since wab dev runs in a web browser, should a clear my cache/history between updates or changes in the code? I remember seeing the Add Tile Layer option, but I didn't test. Maybe browser caching was the issue?

@wspiking
Copy link
Author

wspiking commented Feb 5, 2015

Hi Robert, I'm still seeing the same result. It might be good if someone else tested as well. Here are my steps.

  1. Cleared my browsers history/cache
  2. Started w/ a new instance of wab dev
  3. Re-installed LocalLayer 1.3 in wab dev
  4. Tested in Chrome and IE 10

capture

@rscheitlin
Copy link
Collaborator

Bill,
OK, I have read your comment closer now. "Also, the hard return is just being added to the first line" the <br> that you are seeing in the first line is likely one that you manually added in the text. The widget converts the TeaxtArea dijits \n (hard breaks) behind the scenes into <br>, you will not see this in the popup. When you say the insert image and hyperlink do not work can you go into more specific detail?

@wspiking
Copy link
Author

wspiking commented Feb 5, 2015

In the example above, I'm not adding the
in the first line. It is automatically being added when I hit return.....so the updates you made in 1.3 seem to be working only for the first line of text. The Insert Link and Insert Image buttons do nothing for me when clicked. I've now tested on two different machines with the same result.

@rscheitlin
Copy link
Collaborator

Bill,

you are saying that when you click the button you do not get a popup at all? Can you hit F12 key and tell me what the console is saying?

💡 I would love to hear from someone else if they are seeing this as thus far I can not reproduce what you are seeing.

There is nothing in the code that would insert the <br> in the text window, so I don't understand what you are seeing concerning the first line...

@wspiking
Copy link
Author

wspiking commented Feb 5, 2015

I'm not getting a pop-up at all.

capture

@rscheitlin
Copy link
Collaborator

Bill, are you clicking the popup edit button before you have entered a url for the layer and it gets a little green check mark beside the url? Do you see a list of fields in the popups Key Value tab?

@wspiking
Copy link
Author

wspiking commented Feb 5, 2015

Nope. I'm entering the url and getting the green check mark and the service loads fine within the map. I think we really need to find someone else to test, I don't want to waste your time if it's an issue w/ my setup.

@rscheitlin
Copy link
Collaborator

Bill, I can get the same exact error and the buttons not doing anything, but only if the url for the layer has not been entered and/or verified before I click the popup edit button (pencil).

@rscheitlin
Copy link
Collaborator

Bill,

  1. What layer type are you trying?
  2. Are you adding a new layer or are you editing an existing?
  3. Do you see anything different if you are adding a new layer versus editing and existing?

@WashGIS
Copy link

WashGIS commented Feb 5, 2015

I am still having issues with the insert link and image buttons also. Since I only had a test map in WAB builder any way I got rid of everything and started new. New install of web app builder, new download of Locallayer widget. New map.
­Results

  1.   Added Rest service as tiled service, got green check, clicked configure popup on sublayer, clicked customized, clicked insert link button -> nothing happened, clicked add image -> nothing happened. Opened developer tools. [cid:image001.png@01D04143.0D288D00]
    
  2.   Tried same steps adding as dynamic service ( even though it is a tiled service) same result.
    

From: Robert J Scheitlin [mailto:notifications@github.com]
Sent: Thursday, February 05, 2015 11:15 AM
To: cmndrbensisko/LocalLayer
Cc: Rathvon, Joe
Subject: Re: [LocalLayer] Configure Pop-up: Insert Link and Image Buttons (#7)

Bill,

  1. What layer type are you trying?
  2. Are you adding a new layer or are you editing an existing?
  3. Do you see anything different if you are adding a new layer versus editing and existing?


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-73073856.

@wspiking
Copy link
Author

wspiking commented Feb 5, 2015

Thank you for testing WashGIS.

Robert, I've got to get a deliverable out this afternoon so I won't be able to continue testing until this evening. Thanks again for your help.

@rscheitlin
Copy link
Collaborator

Thanks, guys I can now see the issue and am reopening the bug

@rscheitlin rscheitlin reopened this Feb 5, 2015
@rscheitlin rscheitlin modified the milestones: Version 1.4, Version 1.3 Feb 5, 2015
@WashGIS
Copy link

WashGIS commented Feb 5, 2015

So I played around with adding multiple lines in the popup. I wish I could export my chrome dev log (if you know how let me know) here is a screen shot of the logs. [cid:image001.png@01D04145.CFA8A7F0]

I did a series something like this

  1.   Add dynamic layer to map
    
  2.   Click ok
    
  3.   Open config back up
    
  4.   Open popup config on sublayer
    
  5.   Add text  to custom tab (I didn’t add any html tags just used the return key on the keyboard)
    
  6.   Click ok on popup window
    
  7.   Click ok on layer config and close widget config
    
  8.   Clicked on map and popup displayed just as I wanted – only added two lines and forgot that I wanted to test with more than two lines
    
  9.   Opened back up to popup config
    
  10. Decided to work from empty popup so deleted everything from custom
  11. Ok out and click on map for popup
  12. Still displays fields I added to the custom popup
  13. Reopend popup config
  14. Popup config shows fields checked on (I didn’t check anything)
  15. Removed checked fields
  16. Ok out and click on map for popup
  17. Empty. Good!
  18. Reopend popoup config
  19. Add text to custom tab (I didn’t add any html tags just used the return key on the keyboard) (this time four lines)
  20. Ok out and click on map for popup
  21. Displayed as expected
  22. Reopend popup config
  23. “widget” reformatted an added
    for the first return
  24. Clicked ok to close popup config
  25. Reopend popup config and next return has been replace with

It was something like that. Hope it helps and was not too hard to follow.

From: Robert J Scheitlin [mailto:notifications@github.com]
Sent: Thursday, February 05, 2015 9:56 AM
To: cmndrbensisko/LocalLayer
Cc: Rathvon, Joe
Subject: Re: [LocalLayer] Configure Pop-up: Insert Link and Image Buttons (#7)

Bill,

you are saying that when you click the button you do not get a popup at all? Can you hit F12 key and tell me what the console is saying?

[https://assets-cdn.github.com/images/icons/emoji/unicode/1f4a1.png]I would love to hear from someone else if they are seeing this as thus far I can not reproduce what you are seeing.

There is nothing in the code that would insert the
in the text window, so I don't understand what you are seeing concerning the first line...


Reply to this email directly or view it on GitHubhttps://github.com//issues/7#issuecomment-73058877.

@rscheitlin
Copy link
Collaborator

WashGIS,
Those details are extremely helpful in diagnosing the issues I have in the GUI. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants