it won't press it again if you take an image of the inventory check \Find\Images\buttons\inv*.bmp
if you look at the source you can see that it looks to see if the inventory is open and if it doesn't see it then it presses 'i'
Func _openItems()
If Not WinActive($mabititle) And $searchmode = "Client" Then WinActivate($mabititle)
If _itemsOpen() = 0 Then
Send("{i}")
If $summonpet = 1 Then
If Not WinActive($mabititle) And $searchmode = "Client" Then WinActivate($mabititle)
Send("{" & $petinvkey & "}")
EndIf
EndIf
EndFunc ;==>_openItems
;=======================================================================================
Func _itemsOpen()
$tolerance = $tolerance2
If Not $Mode = "" Then
$file = _FileListToArray(@ScriptDir & "\Find\Images\buttons\", "inv*.bmp", 1)
If Not @error Then
For $i = 1 To $file[0]
$result = _ImageSearch(@ScriptDir & "\Find\Images\buttons\" & $file[$i], 0, $x1, $y1, $tolerance)
ToolTip($file[$i] & " = " & $result & " Tolerance = " & $tolerance, 0, @DesktopHeight - 20)
Sleep($shortestdelay)
If $result = 1 Then
Return 1
EndIf
Next
EndIf
Return 0
EndIf
EndFunc ;==>_itemsOpen
and i use these functions on every bot that uses items and most times i will close all windows so they don't interfere with image scanning something on the ground or whatever, and then open inventory again.
so no, it's not counter productive.