Post doesn't exist
Archived Comments (locked)
|
|
The following is an archive of comments made before threaded discussions was implemented (November 16th, 2008) |
|
|
Take your time ms99 and have fun :P |
|
|
@all, Now, I'm really sick about the icon discussions. I will replace all with images made by myself, when I get some time.
Thank you. |
|
|
@phoboz
EDIT:
|
|
|
|
|
|
Hi phob0z, Sorry, the solution presented by you does not work on my computer. This is the reason I used the very complicated one included in the script. It may be because of my configuration.
Thank you. EDIT: I should reboot my computer more often :( You're right, it works now. |
|
|
wow... this script could use some compression for sure. Does it lag up the browser or have a million memory leaks? |
|
|
@ms99
this is the "bug", not a real bug, but it looks weird :P
if (parX) {
var attdefTable = document.createElement("TABLE");
attdefTable.setAttribute('class', 'tbg');
attdefTable.setAttribute('cellspacing', '1');
attdefTable.setAttribute('cellpadding', '2');
var hRow = elem("TR", "");
hRow.setAttribute('class', 'rbg');
var hCell = elem("TD", T('STATISTICS'));
hCell.setAttribute('colspan', '4'); //----Modified
hCell.setAttribute('style', 'font-weight:bold;');
hRow.appendChild(hCell);
attdefTable.appendChild(hRow);
//add the total attack, def_i and def_c power for the selected troops
var bRow = elem("TR", "");
bRow.setAttribute('align', getDocDirection);
var aCell = elem("TD", "<img src='" + image["att_all"] + "'>" + minLink);
aCell.setAttribute('id', "troopsattpower");
aCell.setAttribute('width', "33,3%"); //----Added (bug solution)
var bCell = elem("TD", "<img src='" + image["def_i"] + "'>" + minLink);
bCell.setAttribute('id', "troopsdefipower");
bCell.setAttribute('colspan', "2"); //----Added
bCell.setAttribute('width', "33,3%"); //----Added (bug solution)
var cCell = elem("TD", "<img src='" + image["def_c"] + "'>" + minLink);
cCell.setAttribute('id', "troopsdefcpower");
cCell.setAttribute('width', "33,3%"); //----Added (bug solution)
bRow.appendChild(aCell);
bRow.appendChild(bCell);
bRow.appendChild(cCell);
attdefTable.appendChild(bRow);
/* var bTable = document.createElement("TABLE"); //----New table not nedded
bTable.setAttribute('cellspacing', '1');
bTable.setAttribute('cellpadding', '2');
bTable.setAttribute('style', "width:500px; border-style:none solid solid solid; border-color:#C0C0C0; border-width: 0px 1px 1px 1px");
*/
var dRow = elem("TR", "");
//add the total capacity
var dCell = elem("TD", "<img src='" + image["capacity"] + "'>");
dCell.setAttribute('id', 'troopscapacity');
dCell.setAttribute('align', getDocDirection);
dCell.setAttribute('colspan', "2"); //----Added
dCell.setAttribute('width', '50%');
/* if (getDocDirection == 'right') { //----Ummm dont know what it is, but not nedded :P
dCell.setAttribute('style', 'border-style:none none none solid; border-color:#C0C0C0; border-width: 0px 0px 0px 1px');
} else {
dCell.setAttribute('style', 'border-style:none solid none none; border-color:#C0C0C0; border-width: 0px 1px 0px 0px');
}
*/
//add the crop consumption
var eCell = elem("TD", "<img src='" + image["img5"] + "'>");
eCell.setAttribute('id', 'troopscropconsumption');
eCell.setAttribute('align', getDocDirection);
eCell.setAttribute('colspan', "2"); //----Added
eCell.setAttribute('width', '50%');
dRow.appendChild(dCell);
dRow.appendChild(eCell);
attdefTable.appendChild(dRow); //----Modified
//parX.appendChild(attdefTable);
var aDiv = elem("DIV", "");
var pX = elem("P", "");
aDiv.appendChild(pX);
aDiv.appendChild(attdefTable);
// aDiv.appendChild(bTable); //----Not needed
insertAfter(parX, aDiv);
}
|
|
|
haha .. i didnt steal i just showed you how YOUR icon looked like in small size...
about the speed icon.. i didnt copy paste it.. i drew it in my photoshop.. which wasnt easy...
|
|
|
nice touch with the bullet-info. About the capacity-icon. It is not because im "mad" that my icon isn't used. Actually, i dont care. But i thought that in the beginning, the reason we didnt use If we are going to steal, i think 1 note more..
|
|
|
Nicely done ms99 :P How about adding "." if number is more then 1000 on "Total attack/def power/crop consumption/capacity table" -> 1000 would be -> 1.000 (= BmW |
|
|
For space between menu and table... it's not so easy (left menu, villages number, notebook, bookmarks)... All that put the array back
For update table, i understand... but it's not easy for script to see buildings that are being upgrading... ("Je ne parle pas aux cons, ça les instruit" [Audiard]) |
|
|
gdgp -
|
|
|
First of all thanks for the fix on the city walls calculation, was just about to make a screenshot when updated and saw u guys fixed it :) @promprom 1-Ok, so now what I was talking about the space between menus and update tables, here is an example: Travian Beyond - All languages script version: http://i38.tinypic.com/2rwntz7.png
As you can see there the all languages is perfectly justified, which is pretty neat and spares you to have to scroll down each time you want to see the update table, while the ML&CN is not, and it doesn't seem to be the menu, it's just blank space (maybe there is some html/css element wrongly filling that space?).This is nothing really big, but I actually have a friend that loved some of the features of this script but doesn't use because of this :P 2- about the update table I think you didn't really understand what I was telling, I'll describe the situation: I have my barracks at level 6 and update table displays level 6, that's perfect for me. Then issue the upgrade order, so now my barracks are upgrading to level 7, but when I go and see the update table it still shows "barracks level 6-> Extension available", and I don't think this is correct way, as I won't be able to do this upgrade or might even lack the resources for the next level of the barracks (In case I don't watch the numbers, and think that it's available). So I think the table should show the next level of that building in the update table, meaning that it considered a building under upgrade already at the level to which it's being upgraded. Thanks for all the work and feedback :) |
|
|
@all, 1. About having attack/reinforcement icons in the bookmarks on the right side... The answer is NO ! Reason: links to buildings contain normally the "id=" string. And the "id" is different from the "gid". The "id" specifies the position of a building on the map inside the village, without information of what the kind of building it is (and users position their buildings different on the map inside the village). The "gid" specifies only what type of building it is, without specifying the position on the map inside the village. Normal links use the "id=" string. Links using the "gid=" string are not usual (but acceptable as you may create your own browser links). Let's keep the bookmarks simple ! 2. About editing bookmarks: I will not program a general bookmark manager as it is completely out of scope of this script ! You can edit your bookmarks via the Setup page.
3. About locking the bookmarks: The "add/add current page/add spacer/lock & unlock icon" will remain as they are. By locking the bookmarks I understand moving/deleting/changing the existing ones. Adding a bookmark should always be available ! 4. Message link/link to world analyser/link to map will no longer be available on the page where you have a listing of the reports. Reason: you may enable more than one page of reports to be retrieved from the server (which is done via AJAX requests and processing in the background). As I do not see any reason for not being consistent on this page and because inserting those icons for pages processed in the background would require a complete redesign of the functions that add those links, I will just disable the functions while on the "reports listing page". Thank you. (This post may be further processed as I read your comments.) |
|
|
Promprom,
|
|
|
Another option for the crops consumption:
|
|
|
@Promprom: Excuse me ... I can not understand ...
|
|
|
We cant pass action (scout, attack,def) thru URL then we only can put link to "send troop" and "market"...
@All : please edit your comments instead of create a new one (and delete them when they are useless) |
|
|
@DMaster If I can say, I believe that no without in-line is more beautiful. @Promprom I think I understand what you said, but my English is really poor.
|
|
|
I've found the bug for city wall (numbers of buildingcode inverted into buildingcost), I send the bugfix by mail. @All If u know other little bugs tell it to me, i like fix them ;) |
|
|
@BmW
|
|
|
@phob0z
I dont like it... |
|
|
|
|
|
promprom
|



as capacity-icon, was because we didnt make it our self. And now, the current capacity icon
isnt made by DMaster, but stolen from: 



