Improvements to embedding

7 years ago · ad40e5c59a
parent 5a6de74105
commit ad40e5c59a
6 changed files with 9 additions and 5 deletions
--- a/static/editor.js
+++ b/static/editor.js
@ -78,6 +78,7 @@ define(function (require) {
            matchBrackets: true,
            useCPP: true,
            dragDrop: false,
+            readOnly: !!state.options.readOnly,
            extraKeys: {"Alt-F": false}, // see https://github.com/mattgodbolt/gcc-explorer/pull/131
            mode: cmMode
        });
--- a/static/embed.html
+++ b/static/embed.html
@ -11,7 +11,7 @@
 <body class="embedded">

 <a href="/" class="float-link link" target="_blank">
-    Compiler Explorer &dash; <span class="language-name"></span>
+    Edit on <span class="language-name"></span> Compiler Explorer
    <span class="glyphicon glyphicon-new-window"></span><br/>
 </a>

@ -31,7 +31,7 @@
                </button>
            </div>
        </div>
-        <textarea>// Type your code here, or load an example.</textarea>
+        <textarea></textarea>
    </div>

    <div id="compiler">
--- a/static/explorer.css
+++ b/static/explorer.css
@ -19,12 +19,12 @@
    font-size: x-small;
    font-weight: bold;
    text-align: center;
-    background-color: rgba(128,128,128,0.5);
+    background-color: rgba(128, 128, 128, 0.5);
    border-radius: 5px;
 }

 .float-link:hover {
-    background-color: rgba(128,128,128,0.9);
+    background-color: rgba(128, 128, 160, 0.5);
 }

 .template {
--- a/static/index.html
+++ b/static/index.html
@ -13,7 +13,7 @@
 <div class="navbar navbar-inverse">
    <div class="container-fluid">
        <div class="navbar-header">
-            <a class="navbar-brand" href="#">Compiler Explorer - <span class="language-name"></span></a>
+            <a class="navbar-brand" href="#">Compiler Explorer &dash; <span class="language-name"></span></a>
        </div>
        <li class="navbar-collapse collapse">
            <ul class="nav navbar-nav navbar-left">
--- a/static/main.js
+++ b/static/main.js
@ -109,6 +109,8 @@ define(function (require) {
                options = state.options;
                filters = _.extend(filters, state.filters);
            });
+        if (!filters.compileOnChange)
+            filters.readOnly = true;
        return window.location.origin + '/e#' + url.risonify({
                filters: _.keys(filters).join(","),
                source: source,
--- a/test/embedding.html
+++ b/test/embedding.html
@ -13,4 +13,5 @@
        src="http://lud-ldnmg01:10240/e#source:'int+main()%7B%7D',compiler:g44,options:'-O2',filters:'colouriseAsm,intel,commentOnly,directives,labels,compileOnChange'"></iframe>
 </body>
 <div>And here's some text after</div>
+<iframe width="800px" height="200px" src="http://lud-mgodbolt01:10240/e#compiler:/usr/bin/g%2B%2B,filters:'colouriseAsm,readOnly,labels,directives,commentOnly,intel',options:'-O2',source:'//+Type+your+code+here,+or+load+an+example.%0Aint+square(int+num)+%7B%0A++++return+num+*+num%3B%0A%7D%0A'"></iframe>
 </html>