Replace complex Job Specs page with minimal HTML page and alert button#1
Open
PreetM07 wants to merge 1 commit into
Open
Replace complex Job Specs page with minimal HTML page and alert button#1PreetM07 wants to merge 1 commit into
PreetM07 wants to merge 1 commit into
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 631a5dd714
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Comment on lines
-6
to
+10
| </p> | ||
| </div> | ||
| </div> | ||
| <div class="card" id="si-Grammar&Punctuation-Weights" primarysort="2" secondarysort="Style_Guide"> | ||
| <h5 class="alert alert-success"> | ||
| <a data-toggle="collapse" data-target="#si-Weights" href="javascript:void(0);"> | ||
| <span class="badge role-icon badge-success">Style Guide</span> | ||
| <span>Weights</span> | ||
| </a> | ||
| </h5> | ||
| <div id="si-Weights" class="card-body card-normal collapse"> | ||
| <p> | ||
| <table width="100%" cellpadding="2" class="table table-condensed table-bordered"> | ||
| <th colspan="2">Job Spec</th> | ||
| <tr> | ||
| <td> | ||
| <p>Weights are typed as signs and numerals, not words. For example: 5g instead of five grams.</p> | ||
| </td> | ||
| </tr> | ||
| </table> | ||
| </p> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| <div id="tab-SupportMaterial" style="padding-bottom: 40px;"> | ||
| <h3 class="cathead">Support Material</h3> | ||
| <p>Any job specs that relate specifically to support material can be found in this section.</p> | ||
| <div class="catcards"> | ||
| <div class="card" id="si-SupportMaterial-Research" primarysort="2" secondarysort="Style_Guide"> | ||
| <h5 class="alert alert-success"> | ||
| <a data-toggle="collapse" data-target="#si-Research" href="javascript:void(0);"> | ||
| <span class="badge role-icon badge-success">Style Guide</span> | ||
| <span>Research</span> | ||
| </a> | ||
| </h5> | ||
| <div id="si-Research" class="card-body card-normal collapse"> | ||
| <p> | ||
| <table width="100%" cellpadding="2" class="table table-condensed table-bordered"> | ||
| <th colspan="2">Job Spec</th> | ||
| <tr> | ||
| <td> | ||
| <p>If the spelling of a name, word, phrase or specialist term is unsure, then it needs verifying with an internet search. Highlight it by writing it in [].</p> | ||
| </td> | ||
| </tr> | ||
| </table> | ||
| </p> | ||
| <div class="troubleshootingopen"> | ||
| <h5 style="padding-top: 30px;" class="troubleshootingheader">Troubleshooting tips:</h5> | ||
| <table width="100%" cellpadding="2" class="table table-condensed table-bordered"> | ||
| <tr> | ||
| <th class="troubleshooting" width="180px">Problem</th> | ||
| <th class="troubleshooting">Solution</th> | ||
| </tr> | ||
| <tr> | ||
| <td class="troubleshooting">Quick Google check</td> | ||
| <td class="troubleshooting">If there is a name, word, phrase or specialist term that needs varifying with an internet search, then it is easy to search for the word by highlighting it.</td> | ||
| </tr> | ||
| </table> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| <div class="card" id="si-SupportMaterial-Usingfacesheetsandsupportmaterial" primarysort="2" secondarysort="Style_Guide"> | ||
| <h5 class="alert alert-success"> | ||
| <a data-toggle="collapse" data-target="#si-Usingfacesheetsandsupportmaterial" href="javascript:void(0);"> | ||
| <span class="badge role-icon badge-success">Style Guide</span> | ||
| <span>Using face sheets and support material</span> | ||
| </a> | ||
| </h5> | ||
| <div id="si-Usingfacesheetsandsupportmaterial" class="card-body card-normal collapse"> | ||
| <p> | ||
| <table width="100%" cellpadding="2" class="table table-condensed table-bordered"> | ||
| <th colspan="2">Job Spec</th> | ||
| <tr> | ||
| <td> | ||
| <p>If you have been provided support material, then ensure that it is used when transcribing the media file.</p> | ||
| <p>Support material is provided to us by the client, it is not created by eLanguageWorld.</p> | ||
| <p>Support material can come in many forms, but the most common type of support material provided for pre-edit jobs is the face sheet.</p> | ||
| <p>If a face sheet has been provided, ensure that your script's speaker labels reflect the spellings of the names in the face sheet.</p> | ||
| <p>Also, people often refer to other speakers in their dialogue. When this happens, make sure that use the correct spelling according to the face sheet. However, if someone refers to somebody by their nickname in the dialogue, but the source script only has their actual name, make sure you type what was actually said. Make an educated guess to the spelling of the nickname.</p> | ||
| <p>If support material has been provided, the client expects it to be visible and referred to whilst typing the script. Consider opening the face sheet on a second screen or printing it out in order to fulfil this expectation. Mistakes found in a script that could have been avoided by referring to the support material will be perceived as negligent and inattentive by the client.</p> | ||
| </td> | ||
| </tr> | ||
| </table> | ||
| </p> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| <div id="tab-Admin" style="padding-bottom: 40px;"> | ||
| <h3 class="cathead">Admin</h3> | ||
| <p>Any job specs that relate specifically to admin and availability can be found in this section.</p> | ||
| <div class="catcards"/> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| </div> | ||
| <script>$(document).ready(function () {$("#sidebar").mCustomScrollbar({ theme: "minimal" }); $('#sidebarCollapse').on('click', function () { $('#sidebar, #content').toggleClass('active'); $('.collapse.in').toggleClass('in'); $('a[aria-expanded=true]').attr('aria-expanded', 'false'); }); });</script> | ||
| <h1>Welcome to My Simple Page</h1> | ||
| <button id="alertButton" type="button">Show Alert</button> |
There was a problem hiding this comment.
This change turns index.html into a standalone alert demo and removes the entire Job Specs interface (all sections, navigation, and supporting asset wiring), so any user opening the main page will no longer be able to access the product’s actual content. Because index.html is the repository’s primary entrypoint, this is a release-blocking functional regression rather than a cosmetic simplification; the demo should live in a separate file instead of replacing the app page.
Useful? React with 👍 / 👎.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
index.htmlthat included many assets and app logic with a compact, self-contained file for simple demo or testing use.Description
index.htmlcontents with a minimal HTML document including<!DOCTYPE html>,metacharset and viewport, and atitleofSimple Alert Page.buttonwithid="alertButton"wired to a JavaScript event listener that callsalert('Button clicked!')on click.Testing
nl -ba index.htmlto confirm the expected HTML and JavaScript are present and correct.git status --shortto confirmindex.htmlwas updated in the working tree.Codex Task