views:

2884

answers:

5

Hi all.

I have run in to a bit of a problem and I have done a bit of digging, but struggling to come up with a conclusive answer/fix.

Basically, I have some javascript (created by a 3rd party) that does some whizzbang stuff to page elements to make them look pretty. The code works great on single pages (i.e. no master), however, when I try and apply the effects to a content page within a master, it does not work.

In short I have a master page which contains the main script reference. All pages will use the script, but the parameters passed to it will differ for the content pages.

Master Page Script Reference

<script src="scripts.js" language="javascript" type="text/javascript" />

Single Page

<script>
    MakePretty("elementID");
</script>

As you can see, I need the reference in each page (hence it being in the master) but the actual elements I want to "MakePretty" will change dependant on content.

Content Pages

Now, due to the content page not having a <head> element, I have been using the following code to add it to the master pages <head> element:

HtmlGenericControl ctl = new HtmlGenericControl("script");
ctl.Attributes.Add("language", "javascript");
ctl.InnerHtml = @"MakePretty(""elementID"")";
Master.Page.Header.Controls.Add(ctl);

Now, this fails to work. However, if I replace with something simple like alert("HI!"), all works fine. So the code is being added OK, it just doesn't seem to always execute depending on what it is doing..

Now, having done some digging, I have learned that th content page's Load event is raised before the master pages, which may be having an effect, however, I thought the javascript on the page was all loaded/run at once?

Forgive me if this is a stupid question, but I am still relatively new to using javascript, especially in the master pages scenario.

How can I get content pages to call javascript code which is referenced in the Master page?

Thanks for any/all help on this guys, you will really be helping me out with this work problem.

NOTES:

  • RegisterStartupScript and the like does not seem to work at any level..
  • The control ID's are being set fine, even in the MasterPage environment and are rendering as expected.


Apologies if any of this is unclear, I am real tired so if need be please comment if a re-word/clarification is required.

+2  A: 

Isn't it possible to do with clean javascript ?-)

-- just add something similar to this inside the body-tag:

<script type="text/javascript">
  window.onload = function(){
    MakePretty("elementID");
  }
</script>

By the way the script-tag has to have an end-tag:

<script type="text/javascript" src="myScript.js"></script>
roenving
+4  A: 

Put a ContentPlaceHolder in the head section of the master page, then add a asp:Content control on the content page referring to the placeholder and put your script in that control. You can customize it for each page this way.

Also, the reference by ID may not be working because when you use Master Pages, the control IDs on the page are automatically created based on the container structure. So instead of "elementID" as expected, it may be outputting "ctl00_MainContentPlaceHolder_elementID" View your source or use firebug to inspect your form elements to see what the IDs outputted are.

John Sheehan
This tip really helped lead me down a nice path towards getting this done, as well as making the code much more elegant in re-use (i.e. the ref in the master page and being able to make customised calls based on content. Thanks, I appreciate it!
Rob Cooper
A: 

Once you use Master Pages, the ids of controls on the client side aren't what you think they are. You should use Control.ClientID when you generate the script.

Lou Franco
Question updated to include point on ID's, these are being set fine.
Rob Cooper
+1  A: 

Why not use jQuery to find all the controls? Something like this:

$(document).ready(function(){
  $("input[type='text'], input[type='radio'], input[type='checkbox'], select, textarea").each(function(){
    MakePretty(this);
  });
});

This way you'll get all elements on the page, you can wait until the page is ready (so you don't modify the DOM illigally). The jQuery selector can get the elements in a bit more of a specific format if you need (ie, add a root element, like the ID of the body div).

It'd also be best to modify the MakePretty method so it takes the element not the ID as the parameter to reduce processing overhead.

Slace
A: 

When using master pages, you need to be careful with the html attribute ID, since .NET will modify this value as it needs to keep ids unique.

I would assume your javascript is applying css styles via ID, and when you are using master pages the ID is different than what is in your aspx. If you verify your javascript is always being added, your answer needs to take into account the following:

  • ALWAYS set your master page id in page load (this.ID = "myPrefix";)
  • Any HTML element in your master page will be prefixed by the master page id (i.e.: on the rendered page will be "myPrefix_myDiv")
  • Any HTML element in your content place holder id will be prefixed with an additional prefix (i.e. myPrefix_ContentPlaceHolderId1_myDiv)

Please let me know if I can clarify anything. Hope this helps!

Mario